Opened 7 years ago

Closed 7 years ago

#1295 closed defect (fixed)

bit_depth_chroma_minus8 syntax inconsistent in monochrome SPS between HM and Text

Reported by: marcin.jasinski Owned by:
Priority: major Milestone: HM+RExt-8.0
Component: HM RExt Version: RExt-7.2 (HM-14.0)
Keywords: Cc: davidf, joel, teruhiko, jct-vc@…

Description

In Text bit_depth_chroma_minus8 is supposed to be alwas present when in HM there is a condition for monochrome and it seems to be proper:
encoder:

if (chromaEnabled)
{
    WRITE_UVLC( chromaEnabled ? (pcSPS->getBitDepth(CHANNEL_TYPE_CHROMA) - 8):0,  "bit_depth_chroma_minus8" ); // NOTE: RExt - This SE is not in the SPS header for 4:0:0.
}

decoder:

const Bool bChroma=(pcSPS->getChromaFormatIdc() != CHROMA_400);
if (bChroma)
{
    READ_UVLC( uiCode,    "bit_depth_chroma_minus8" ); // NOTE: RExt - This SE is not in the SPS header for 4:0:0
}

Change History (3)

comment:1 Changed 7 years ago by DefaultCC Plugin

  • Cc davidf joel teruhiko jct-vc@… added

comment:2 Changed 7 years ago by davidf

  • Component changed from RExt text to HM RExt
  • Milestone set to HM+RExt-8.0
  • Version changed from RExt D7 (Q1005) v8 to RExt-7.2 (HM-14.0)

The SPS and PPS have been designed such that there is no requirement to interpret the profile in order to parse them.

The software should be adjusted to match the text.

comment:3 Changed 7 years ago by karlsharman

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

Fixed in r3988.

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

  • David Flynn(Subscriber, Participant)
  • jct-vc@…(Subscriber)
  • Joel(Subscriber)
  • Karl Sharman(Participant)
  • karl.sharman@…(Always)
  • Karsten Suehring(Always)
  • Marcin Jasiński(Reporter)
  • Teruhiko Suzuki(Subscriber)