Opened 11 years ago

Closed 6 years ago

#1081 closed enhancement (wontfix)

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 (2)

comment:1 Changed 11 years ago by DefaultCC Plugin

  • Cc bbross wjhan jct-vc@… added

comment:2 Changed 6 years ago by bbross

  • Resolution set to wontfix
  • Status changed from new to closed

The redundant condition is harmless.

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, Participant)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Always)
  • Peter de Rivaz(Reporter)
  • Woo-Jin Han(Subscriber)