Opened 4 years ago

Closed 4 years ago

#1221 closed defect (fixed)

Text/SW mismatch CIIP deblocking (8.3.3.5)

Reported by: fbossen Owned by:
Priority: minor Milestone:
Component: spec Version: VVC D10 vD
Keywords: Cc: ksuehring, bbross, XiangLi, fbossen, jvet@…

Description

In the presence of CIIP-coded blocks, boundary strength setting is governed by:
– Otherwise, if the block edge is also a coding block edge and the sample p0 or q0 is in a coding block with ciip_flag equal to 1, bS[ xDi ][ yDj ] is set equal to 2.

This appears to be inconsistent with the SW, which I believe does not have the restriction on the block edge being a coding block edge. For example, in the case where the max transform size is 32 and a CU is larger than 32 in either dimension, there are transform block edges internal to the CU, and as far as I can tell, the SW assigns a value 2 to bS for these internal edges if the CU uses CIIP.

To align with the software, the text should be modified as:
– Otherwise, if the sample p0 or q0 is in a coding block with ciip_flag equal to 1, bS[ xDi ][ yDj ] is set equal to 2.

Attachments (1)

ciip3.bin (61.8 KB) - added by fbossen 4 years ago.
VTM9 bitstream with CIIP and max transform size set to 32

Download all attachments as: .zip

Change history (5)

comment:1 Changed 4 years ago by jlchen

It sounds reasonable to me matching the specification text to software for this issue. Can other deblocking experts confirm this change?

Changed 4 years ago by fbossen

VTM9 bitstream with CIIP and max transform size set to 32

comment:2 Changed 4 years ago by fbossen

Since none of the VTM9 conformance streams seem to exercise this case, a bitstream that does is provided.

comment:3 Changed 4 years ago by Kenneth

Good catch Frank. Make sense to align text with the software as you suggest.

comment:4 Changed 4 years ago by bbross

  • Resolution set to fixed
  • Status changed from new to closed

Thanks! This will be fixed in JVET-S2001-vE.

Note: See TracTickets for help on using tickets.