Opened 10 years ago

Closed 10 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 10 years ago by DefaultCC Plugin

  • Cc davidf karlsharman jct-vc@… added

comment:2 Changed 10 years ago by karlsharman

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

comment:3 Changed 10 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 10 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)