Opened 6 years ago
Closed 6 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@… |
Description
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);
Best.
Jane
Change history (1)
comment:1 Changed 6 years ago by XiangLi
- Milestone set to VTM-4.1
- Resolution set to fixed
- Status changed from new to closed
Fixed in https://vcgit.hhi.fraunhofer.de/jvet/VVCSoftware_VTM/merge_requests/359.