Opened 5 years ago
Closed 5 years ago
#430 closed defect (fixed)
Issue in condition to parse lfnst_idx when treeType = DUAL_TREE_CHROMA
Reported by: | forayr | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | VTM-6.0 |
Component: | VTM | Version: | VTM-5.2 |
Keywords: | Cc: | ksuehring, XiangLi, fbossen, jvet@… |
Description
This concern the following condition:
if( cu.ispMode != NOT_INTRA_SUBPARTITIONS || #if JVET_O0925_MIP_SIMPLIFICATIONS (cu.cs->sps->getUseLFNST() && CU::isIntra(cu) && cu.mipFlag && !allowLfnstWithMip(cu.firstPU->lumaSize())) || #else cu.mipFlag == true || #endif ( CS::isDualITree( *cu.cs ) && cu.chType == CHANNEL_TYPE_CHROMA && std::min( cu.blocks[ 1 ].width, cu.blocks[ 1 ].height ) < 4 ) #if JVET_O0213_RESTRICT_LFNST_TO_MAX_TB_SIZE #if JVET_O0545_MAX_TB_SIGNALLING || ( cu.blocks[ 0 ].width > cu.cs->sps->getMaxTbSize() || cu.blocks[ 0 ].height > cu.cs->sps->getMaxTbSize() ) #else || ( cu.blocks[ 0 ].width > MAX_TB_SIZEY || cu.blocks[ 0 ].height > MAX_TB_SIZEY ) #endif #endif ) { return; }
When CS::isDualITree( *cu.cs ) && cu.chType == CHANNEL_TYPE_CHROMA, ie treeType = DUAL_TREE_CHROMA, the data in cu.blocks[0] are not valid. For exemple, in this case, cu.blocks[ 0 ].width is equal to 0. Information from cu.blocks[1] should be used.
Change history (2)
comment:1 Changed 5 years ago by forayr
comment:2 Changed 5 years ago by ksuehring
- Milestone set to VTM-6.0
- Resolution set to fixed
- Status changed from new to closed
The fix was merged.
Note: See TracTickets for help on using tickets.
Proposed fix: https://vcgit.hhi.fraunhofer.de/jvet/VVCSoftware_VTM/merge_requests/806