Opened 5 years ago

Last modified 5 years ago

#84 new clean up required

sdcEnableFlag is not aligned with the SW

Reported by: kaizhang Owned by: tech
Priority: minor Component: HTM software
Version: HTM-12.2 Keywords: SDC
Cc: tech, jct-3v@…

Description

In I.7.4.9.5
The variable sdcEnableFlag is derived as specified in the following:
– If CuPredMode[ x0 ][ y0 ] is equal to MODE_INTER, sdcEnableFlag is set equal to ( inter_sdc_flag[ nuh_layer_id ] && PartMode = = PART_2Nx2N )
– Otherwise, if CuPredMode[ x0 ][ y0 ] is equal to MODE_INTRA,
sdcEnableFlag is set equal to ( intra_sdc_dmm_wfull_flag[ nuh_layer_id ] && PartMode[ x0 ][ y0 ] = = PART_2Nx2N )

should be changed to
The variable sdcEnableFlag is derived as specified in the following:
– If CuPredMode[ x0 ][ y0 ] is equal to MODE_INTER, sdcEnableFlag is set equal to ( inter_sdc_flag[ nuh_layer_id ] && PartMode = = PART_2Nx2N )
– Otherwise, if CuPredMode[ x0 ][ y0 ] is equal to MODE_INTRA, sdcEnableFlag is set equal to PartMode[ x0 ][ y0 ] = = PART_2Nx2N.

Change history (3)

comment:1 Changed 5 years ago by DefaultCC Plugin

  • Cc tech jct-3v@… added

comment:2 Changed 5 years ago by tech

  • Component changed from 3D-HEVC text to HTM software
  • Version changed from 3D-HEVC Draft Text 5 to HTM-12.2

This is actually a software bug. Please see decision on JCT3V-I0085 in the Sapporo meeting notes.

comment:3 Changed 5 years ago by tech

  • Type changed from defect to clean up required

Software seems to work also correctly. However, names need to be aligned with spec to avoid misunderstandings.

Hence: getVpsDepthModesFlag() --> getIntraSdcDmmWFullFlag()

Note: See TracTickets for help on using tickets.

This list contains all users that will be notified about changes made to this ticket.

These roles will be notified: Reporter, Owner, Subscriber, Participant

  • Gerhard Tech(Owner, Subscriber, Participant, Always)
  • jct-3v@…(Subscriber)
  • Kai Zhang(Reporter)
  • Karsten Suehring(Always)