Opened 5 years ago
Closed 5 years ago
#1028 closed defect (fixed)
NumEntryPoints in R0481
Reported by: | Tomohiro Ikai | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | VVC D7 vC |
Component: | spec | Version: | VVC D8 vB |
Keywords: | Cc: | ksuehring, bbross, XiangLi, fbossen, jvet@… |
Description
In NumEntryPoints derivation, "&& sps_wpp_entry_point_offsets_present_flag" was removed in R0481. But I thinks just replace "&& sps_wpp_entry_point_offsets_present_flag" with "&& sps_entropy_coding_sync_enabled_flag" should be right description in this case.
Another issue can be gating flag (in R0481 sps_entry_point_offsets_present_flag in SPS while in P2001 entry_point_offsets_present_flag in PPS) but this could be beyond my report.
Change history (4)
comment:1 Changed 5 years ago by yk
comment:2 Changed 5 years ago by Tomohiro Ikai
Thanks, but it seems JVET-R2001-v1 has the same issue in R0481.
The point is that "ctbAddrY != prevCtbAddrY" should only apply to wavefront as follows.
NumEntryPoints = 0
if( sps_entry_point_offsets_present_flag )
...
( ctbAddrY != prevCtbAddrY && sps_entropy_coding_sync_enabled_flag ) )
NumEntryPoints++
comment:3 Changed 5 years ago by yk
Got you. You are right. Added " && sps_entropy_coding_sync_enabled_flag" in R2001-v2 that's being worked on. Thanks!
comment:4 Changed 5 years ago by yk
- Resolution set to fixed
- Status changed from new to closed
Thanks! Pleas take a look at JVET-R2001-v1, and let me know if there is still anything that should be changed regarding this item.