Opened 4 years ago

Closed 4 years ago

#1302 closed defect (fixed)

Table 125 - Context initialization table for abs_level_gtx_flag in wrong order

Reported by: bheng Owned by:
Priority: minor Milestone:
Component: spec Version: VVC D10 vE
Keywords: Cc: ksuehring, bbross, XiangLi, fbossen, jvet@…

Description

I believe the context initialization table for abs_level_gtx_flag has the gt1 and gtx contexts reversed.

The table in the text follows VTM which adds the gtx contexts first, and the gt1 contexts second. However, the context derivation in Section 9.3.4.2.9 assumes the gt1 contexts are first and the gtx contexts are second.

Specifically, for I-slices, table entries [0-31] and [32-63] should be swapped. And, for transform skip, table entries [64-67] and [68-71] should be swapped.

Similarly, the same changes for P-slices and B-slices would be required as well.

Change history (2)

comment:1 Changed 4 years ago by fbossen

The suggested fix looks correct to me.

comment:2 Changed 4 years ago by bbross

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

Good catch, thanks!
This will be fixed in JVET-S2001-vH by swapping:
[0-31] and [32-63]
[64-67] and [68-71]

[72-103] and [104-135]
[136-139] and [140-143]

[144-175] and [176-207]
[208-211] and [212-215]

Also, I fixed the issue that indices 128..143 appared twice in the index row while 144..159 are missing.

Note: See TracTickets for help on using tickets.