Opened 10 years ago

Closed 8 years ago

#208 closed defect (fixed)

Wrong number of contexts defined for inter prediction direction

Reported by: fbossen Owned by:
Priority: major Milestone: HM-4.1
Component: HM Version: HM-4.0
Keywords: Cc: fbossen, ksuehring, davidf, jct-vc@…

Description

The number of contexts for coding inter prediction direction is defined as:
#define NUM_INTER_DIR_CTX 4 /< number of context models for inter prediction direction

When DNB_INTER_PRED_MODE is defined, there are actually 5 contexts being used in TDecSbac::parseInterDir:

ContextModel *pCtx = m_cCUInterDirSCModel.get( 0 );

...

pCtx++;
m_pcTDecBinIf->decodeBin( uiSymbol, *( pCtx + 3 ) );

This fifth context, not being properly defined, is aliased with the first context of m_cCURefPicSCModel

Change History (4)

comment:1 Changed 10 years ago by davidf

  • Component set to HM

Updating component after adding WD (Text) tickets

comment:2 Changed 10 years ago by davidf

  • Cc fbossen ksuehring davidf added

comment:3 Changed 10 years ago by davidf

  • Cc jct-vc@… added

comment:4 Changed 8 years ago by fbossen

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

Appears to be fixed in recent HM versions (NUM_INTER_DIR_CTX is set to 5)

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