Opened 7 years ago

Closed 7 years ago

#1269 closed defect (fixed)

initValues for cu_chroma_qp_adjustment_flag and cu_chroma_qp_adjustment_idc differ from spec

Reported by: jackh Owned by:
Priority: minor Milestone: HM+RExt-4.2
Component: HM RExt Version: RExt D6 (P1005) v1
Keywords: Cc: davidf, karlsharman, jct-vc@…

Description

We are currently adding range extensions support to Argon Streams. The head of the HM-range-extensions branch contains the following section in ContextTables.h:

#if RExt__O0044_CU_ADAPTIVE_CHROMA_QP_OFFSET
static const UChar
INIT_CHROMA_QP_ADJ_FLAG[NUMBER_OF_SLICE_TYPES][NUM_CHROMA_QP_ADJ_FLAG_CTX] =
{
  { 154, },
  { 154, },
  { 154, },
};

static const UChar
INIT_CHROMA_QP_ADJ_IDC[NUMBER_OF_SLICE_TYPES][NUM_CHROMA_QP_ADJ_IDC_CTX] =
{
  { 154, },
  { 154, },
  { 154, },
};
#endif

In the spec (P1005 v2) these are given as 153. Could you clarify which is correct?

Change History (4)

comment:1 Changed 7 years ago by DefaultCC Plugin

  • Cc davidf karlsharman jct-vc@… added

comment:2 Changed 7 years ago by karlsharman

  • Version changed from RExt-4.1 (HM-12.0) to RExt D6 (P1005) v1

comment:3 Changed 7 years ago by jackh

We note that the specification has been changed to match the reference code in version Q1005 v5. Presumably this bug can now be closed?

comment:4 Changed 7 years ago by davidf

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

Correct, the text and the software now match.

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)
  • Jack Haughton(Reporter, Participant)
  • jct-vc@…(Subscriber)
  • Karl Sharman(Subscriber, Participant)
  • karl.sharman@…(Always)
  • Karsten Suehring(Always)