id,summary,reporter,owner,description,type,status,priority,milestone,component,version,resolution,keywords,cc 923,Editorial modification: GEO size constraint on 8x64/64x8,chunchic,,"According to the meeting notes which records ""Decision: Stay with the 64-combination method from the CE, except disable GEO for 8x64 and 64x8"", the decision made for GEO CU size constraint is to disable 8x64 and 64x8 CUs. However, the both the spec (verison D) and VTM-8.0 implemeted this adoption differentely as: [JVET-2001-vD] * Section 7.3.10.7: {{{ if( sps_ciip_enabled_flag && sps_gpm_enabled_flag && slice_type = = B && cu_skip_flag[ x0 ][ y0 ] = = 0 && cbWidth >= 8 && cbHeight >= 8 && '''cbWidth < ( 8 * cbHeight ) && cbHeight < ( 8 * cbWidth ) &&''' cbWidth < 128 && cbHeight < 128 ) }}} [VTM-8.0] * CABACReader.cpp at Line 2319 and CABACWriter.cpp at Line 1993: {{{ const bool geoAvailable = pu.cu->cs->slice->getSPS()->getUseGeo() && pu.cu->cs->slice->isInterB() && pu.cu->cs->picHeader->getMaxNumGeoCand() > 1 && pu.cu->lwidth() >= GEO_MIN_CU_SIZE && pu.cu->lheight() >= GEO_MIN_CU_SIZE && pu.cu->lwidth() <= GEO_MAX_CU_SIZE && pu.cu->lheight() <= GEO_MAX_CU_SIZE ''' && pu.cu->lwidth() < 8 * pu.cu->lheight() && pu.cu->lheight() < 8 * pu.cu->lwidth();''' }}} Although CU's width-to-height ratio of either 1:8 or 8:1 covers the 8x64 and 64x8 CUs, these ratios may not be identical to ""8x64/64x8"" that are recorded in the meeting notes. It seems the meeting notes may refer to JVET-Q0123 for adoption, although the it does not record the document number explicitly. To reflect to the adoption of GEO size constraint recorded in the meeting notes, both spec and VTM-8.0 are supposed to be changed based on JVET-Q0123, which is ""( cbWidth + cbHeight ) != 72"". As the spec and VTM software did similarly for disallowing 8x4/4x8 bi-prediction by explicitly specifying ""The value of ( cbWidth + cbHeight ) is equal to 12"", here it may be more adequate to follow this same editorial fashion to disable 8x64/64x8 GEO (that is cbWidth + cbHeight != 72 as in JVET-Q0123), as follows. [JVET-2001-vD] * Section 7.3.10.7: {{{ if( sps_ciip_enabled_flag && sps_gpm_enabled_flag && slice_type = = B && cu_skip_flag[ x0 ][ y0 ] = = 0 && cbWidth >= 8 && cbHeight >= 8 && '''( cbWidth + cbHeight ) != 72 &&''' cbWidth < 128 && cbHeight < 128 ) }}} [VTM-8.0] * CABACReader.cpp at Line 2319 and CABACWriter.cpp at Line 1993: {{{ const bool geoAvailable = pu.cu->cs->slice->getSPS()->getUseGeo() && pu.cu->cs->slice->isInterB() && pu.cu->cs->picHeader->getMaxNumGeoCand() > 1 && pu.cu->lwidth() >= GEO_MIN_CU_SIZE && pu.cu->lheight() >= GEO_MIN_CU_SIZE && pu.cu->lwidth() <= GEO_MAX_CU_SIZE && pu.cu->lheight() <= GEO_MAX_CU_SIZE '''&& (pu.cu->lwidth() + pu.cu->lheight()) != 72;''' }}} ",defect,closed,minor,,spec,VVC D8 vB,worksforme,,ksuehring XiangLi fbossen jvet@… bbross