id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
269	Mismatch between VTM and specification on allowBtSplit derivation	fbarbier		"In spec ** JVET-N1001-v5.docx **, 
6.4.2 Allowed binary split process
{{{
The variable allowBtSplit is derived as follows:
  - If one or more of the following conditions are true, allowBtSplit is set equal to FALSE:
    • cbSize is less than or equal to MinBtSizeY
    • cbWidth is greater than maxBtSize
    • cbHeight is greater than maxBtSize
    • mttDepth is greater than or equal to maxMttDepth""
}}}

In ** VTM **, in file source/Lib/CommonLib/UnitPartitioner.cpp:396, function (QTBTPartitioner::canSplit)

{{{
if( area.height <= minBtSize || area.height > maxBtSize )   canBh = false;
...
if( area.width <= minBtSize || area.width > maxBtSize )     canBv = false;
...
}}}

To match specification, I think it should be :

{{{
  if( area.width > maxBtSize || area.height > maxBtSize )
  {
    canBh = canBv = false;
  }
  ...
  if( area.height <= minBtSize )    canBh = false;
  if( area.width <= minBtSize )     canBv = false;
}}}"	defect	closed	minor	VTM-5.1	VTM	VTM-5.0rc1	fixed		ksuehring XiangLi fbossen jvet@…