Opened 4 years ago

Closed 4 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 4 years ago by yk

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.

comment:2 Changed 4 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 4 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 4 years ago by yk

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.