Opened 4 years ago

Last modified 4 years ago

#1081 new enhancement

Redundant condition in availability calculation

Reported by: peterderivaz Owned by:
Priority: trivial Milestone: HM-10.1
Component: Text Version: D10 (L1003) v33
Keywords: Cc: bbross, wjhan, jct-vc@…


In "6.4.2 Derivation process for prediction block availability" when sameCb is TRUE, the text checks the following conditions:

( nPbW << 1 ) is equal to nCbS,
( nPbH << 1 ) is equal to nCbS,
partIdx is equal to 1,
( yCb + nPbH ) is less than or equal to yNbY,
( xCb + nPbW ) is greater than xNbY.

These conditions correspond to:

The coding block is partitioned into 4 squares
The current part is top right
The neighbour pixel is below the current part
The neighbour pixel is left of the current part

However, I believe that the final condition is redundant. The decode process will never try to predict from locations that are directly below the current part.

I doubt that this actually requires a change, as the current text is entirely correct, but I have posted this bug in case:

  1. Someone may point out that I have misunderstood something and the final condition is important
  2. Someone else is similarly confused by this
  3. It is felt that it would help to modify the text

Change History (1)

comment:1 Changed 4 years ago by DefaultCC Plugin

  • Cc bbross wjhan jct-vc@… added
Note: See TracTickets for help on using tickets.

This list contains all users that will be notified about changes made to this ticket.

These roles will be notified: Reporter, Owner, Subscriber, Participant

  • Benjamin Bross(Subscriber)
  • David Flynn(Always)
  • jct-vc@…(Subscriber)
  • Karl Sharman(Always)
  • Karsten Suehring(Always)
  • Peter de Rivaz(Reporter)
  • Woo-Jin Han(Subscriber)