Opened 5 years ago
Closed 5 years 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 5 years ago by chiaming
- Version changed from VVC D6 vE to VVC D7 vB
comment:2 Changed 5 years ago by bbross
- Resolution set to fixed
- Status changed from new to closed
Thanks for reporting.
This will be fixed JVET-P2001-vC.