Opened 5 years ago
Closed 5 years ago
#446 closed defect (fixed)
Issue in pred_mode_ibc_flag parsing
Reported by: | forayr | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | |
Component: | spec | Version: | VVC D7 vC |
Keywords: | Cc: | ksuehring, bbross, XiangLi, fbossen, jvet@… |
Description
There is a mismatch between the spec and the specification on pred_mode_ibc_flag parsing.
The following modification to the specification are needed to align its behavior with the vtm:
In the Syntax in tabular form:
if( ( ( slice_type == I && cu_skip_flag[ x0 ][ y0 ] == 0 ) || ( slice_type != I && ( CuPredMode[ chType ][ x0 ][ y0 ] != MODE_INTRA || - ( ( cbWidth == 4 && cbHeight == 4 && cu_skip_flag[ x0 ][ y0 ] == 0 ) ) ) ) && + ( ( ( cbWidth == 4 && cbHeight == 4 ) || treeType == MODE_TYPE_INTRA ) && + cu_skip_flag[ x0 ][ y0 ] == 0 ) ) ) ) && cbWidth <= 64 && cbHeight <= 64 && modeType != MODE_TYPE_INTER && sps_ibc_enabled_flag && treeType != DUAL_TREE_CHROMA ) pred_mode_ibc_flag
In the semantics of pred_mode_ibc_flag:
When pred_mode_ibc_flag is not present, it is inferred as follows:
• If cu_skip_flag[ x0 ][ y0 ] is equal to 1, and cbWidth is equal to 4, and cbHeight is equal to 4, pred_mode_ibc_flag is inferred to be equal 1.
+ • Otherwise, if cu_skip_flag[ x0 ][ y0 ] is equal to 1, and modeType == MODE_TYPE_INTRA, pred_mode_ibc_flag is inferred to be equal 1.
• Otherwise, if both cbWidth and cbHeight are equal to 128, pred_mode_ibc_flag is inferred to be equal to 0.
• Otherwise, if modeType is equal to MODE_TYPE_INTER, pred_mode_ibc_flag is inferred to be equal to 0.
• Otherwise, if treeType is equal to DUAL_TREE_CHROMA, pred_mode_ibc_flag is inferred to be equal to 0.
• Otherwise, pred_mode_ibc_flag is infered to be equal to the value of sps_ibc_enabled_flag when decoding an I slice, and 0 when decoding a P or B slice, respectively.
Change history (4)
comment:1 Changed 5 years ago by bbross
comment:2 Changed 5 years ago by zhaoyin
As the proponent of JVET-O0050, I think the suggested spec changes fix a bug of JVET-O0050 spec.
comment:3 Changed 5 years ago by bbross
- Version changed from VVC D6 vE to VVC D7 vC
comment:4 Changed 5 years ago by bbross
- Resolution set to fixed
- Status changed from new to closed
Thanks for reporting.
This will be fixed JVET-P2001-vD.
Note: See TracTickets for help on using tickets.
Can someone confirm that aligning the spec to VTM, i.e. adding the condition on modeType introduced by JVET-O0050, is in the spirit of what we adopted w.r.t JVET-O0050?