#245 closed defect (fixed)
spec, software mismatch of boundary partition for corner case
Reported by: | hangao | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | |
Component: | spec | Version: | VVC D5 v10 |
Keywords: | partition | Cc: | adam.wieckowski@…, ksuehring, bbross, XiangLi, fbossen, jvet@… |
Description
For example, if a 64x64 CU located on the corner (both bottom-left sample and top-right sample are outside of the picture boundaries) and CU size is greater than MinQtSize case
In the software: only QT is allowed, no signaling is needed.
In the spec: split_cu_flag is inferred to be 1, however, split_qt_flag is signaled because both allowSplitQT and allowSplitBtHor are true based on the following table in JVET-N1001-v2 page 42
if( split_cu_flag ) {
if( ( allowSplitBtVer | | allowSplitBtHor | | allowSplitTtVer | | allowSplitTtHor ) && allowSplitQT )
split_qt_flag
A fix could be as following by adding the bold part into 6.4.2:
6.4.2 Allowed binary split process
…
– Otherwise, if all of the following conditions are true, allowBtSplit is set equal to FALSE
– btSplit is equal to SPLIT_BT_HOR
– cbWidth is greater than MaxTbSizeY
– y0 + cbHeight is greater than pic_height_in_luma_samples
– Otherwise, if all of the following conditions are true, allowBtSplit is set equal to FALSE
– x0 + cbWidth is greater than pic_width_in_luma_samples
– y0 + cbHeight is greater than pic_height_in_luma_samples
– cbWidth is greater than minQTSize
– Otherwise, if all of the following conditions are true, allowBtSplit is set equal to FALSE
– btSplit is equal to SPLIT_BT_HOR
– x0 + cbWidth is greater than pic_width_in_luma_samples
– y0 + cbHeight is less than or equal to pic_height_in_luma_samples
…
Thanks Adam for the crosscheck
Change history (8)
comment:1 Changed 5 years ago by bbross
- Version changed from VVC D4 v7 to VVC D5 v4
comment:2 Changed 5 years ago by bbross
- Version changed from VVC D5 v4 to VVC D5 v5
comment:3 Changed 5 years ago by bbross
- Version changed from VVC D5 v5 to VVC D5 v6
comment:4 Changed 5 years ago by bbross
- Version changed from VVC D5 v6 to VVC D5 v7
comment:5 Changed 5 years ago by bbross
- Version changed from VVC D5 v7 to VVC D5 v8
comment:6 Changed 5 years ago by bbross
- Version changed from VVC D5 v8 to VVC D5 v9
comment:7 Changed 5 years ago by bbross
- Resolution set to fixed
- Status changed from new to closed
comment:8 Changed 5 years ago by bbross
- Version changed from VVC D5 v9 to VVC D5 v10
Fixed in D5 v10