Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#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

Fixed in D5 v10

comment:8 Changed 5 years ago by bbross

  • Version changed from VVC D5 v9 to VVC D5 v10
Note: See TracTickets for help on using tickets.