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
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.
The suggested fix looks correct to me.