Opened 4 weeks ago

Closed 6 days ago

#583 closed defect (fixed)

Mismatch between SW and Spec on enabling PDPC

Reported by: chiaming Owned by:
Priority: minor Milestone:
Component: spec Version: VVC D7 vB
Keywords: Cc: ksuehring, bbross, XiangLi, fbossen, jvet@…

Description

In 8.4.5.2.14, the following are used to derive nScale for PDPC:

The variable nScale is derived as follows:
– If predModeIntra is greater than INTRA_ANGULAR50, nScale is set equal to Min( 2, Log2( nTbH ) − Floor( Log2( 3 * invAngle − 2 ) ) + 8 ), using invAngle as specified in clause 8.4.5.2.12.
– Otherwise, if predModeIntra is less than INTRA_ANGULAR18 and not equal to INTRA_PLANAR, nScale is set equal to Min( 2, Log2( nTbW ) − Floor( Log2( 3 * invAngle − 2 ) ) + 8 ), using invAngle as specified in clause 8.4.5.2.12.
– Otherwise, nSacle is set to ( ( Log2( nTbW ) + Log2( nTbH ) − 2 ) >> 2 ).

It should be corrected to

The variable nScale is derived as follows:
– If predModeIntra is greater than INTRA_ANGULAR50, nScale is set equal to Min( 2, Log2( nTbH ) − Floor( Log2( 3 * invAngle − 2 ) ) + 8 ), using invAngle as specified in clause 8.4.5.2.12.
– Otherwise, if predModeIntra is less than INTRA_ANGULAR18, and predModeIntra is not equal to INTRA_PLANAR and INTRA_DC, nScale is set equal to Min( 2, Log2( nTbW ) − Floor( Log2( 3 * invAngle − 2 ) ) + 8 ), using invAngle as specified in clause 8.4.5.2.12.
– Otherwise, nSacle is set to ( ( Log2( nTbW ) + Log2( nTbH ) − 2 ) >> 2 ).

Actually the issue had been reported in #489, but was not completely fixed in the latest spec (i.e., JVET-P2001-v8).

Change history (2)

comment:1 Changed 7 days ago by chiaming

  • Version changed from VVC D6 vE to VVC D7 vB

comment:2 Changed 6 days ago by bbross

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

Thanks for reporting.

This will be fixed JVET-P2001-vC.

Note: See TracTickets for help on using tickets.