Opened 5 years ago

Closed 5 years ago

#217 closed enhancement (fixed)

misalignment between VTM and spec text regarding MRG_MAX_NUM_CANDS

Reported by: LGE_VCC Owned by:
Priority: minor Milestone: VTM-4.1
Component: VTM Version: VTM-4.0.1
Keywords: Cc: ksuehring, XiangLi, fbossen, jvet@…


In VVC4 spec text. MaxNumMergeCand cannot exceed 6. This is specified by
“ MaxNumMergeCand = 6 − six_minus_max_num_merge_cand
The value of MaxNumMergeCand shall be in the range of 1 to 6, inclusive.”

But in VTM4, in commendef.h

static const int MRG_MAX_NUM_CANDS = 7;

This constant number won’t hurt CTC, because in CTC, the MaxNumMergeCand is read from config file and 6 is being used. But this constant definition may cause confusion on what is the max allowed merge list size.

So I think a more consistent implementation would be

static const int MRG_MAX_NUM_CANDS = 6;
static const int MAX_NUM_HMVP_CANDS = (MRG_MAX_NUM_CANDS-1);



Change history (1)

comment:1 Changed 5 years ago by XiangLi

  • Milestone set to VTM-4.1
  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.