Opened 10 years ago

Closed 10 years ago

#73 closed defect (fixed)

A text bug for the post filtering of DBBP

Reported by: zxgvideo Owned by: Xianguo Zhang
Priority: minor Component: 3D-HEVC text
Version: 3D-HEVC Draft Text 4 Keywords: Post filter, DBBP
Cc: tech, jct-3v@…

Description

The current 3D-HEVC text specifies following post filtering conditions for DBBP
if( ( lFlag | | cFlag | | rFlag ) && ( !lFlag | | !cFlag | | !rFlag ) )

filt = ( p[ Max( 0, x − 1 ) ][ y ] + ( filt << 1 ) + p[ Min( x + 1, nCbSX − 1 ) ][ y ] ) >> 2

if( ( tFlag | | cFlag | | bFlag ) && ( !tFlag | | !cFlag | | !bFlag ) )

filt = ( p[ x ][ Max( 0, y − 1 ) ] + ( filt << 1 ) + p[ x ][ Min( y + 1, nCbSX − 1 ) ] ) >> 2

However, the condition in "if" is opposite to the HTM11.0. It should be changed to
if( !(( lFlag | | cFlag | | rFlag ) && ( !lFlag | | !cFlag | | !rFlag ) ) )

filt = ( p[ Max( 0, x − 1 ) ][ y ] + ( filt << 1 ) + p[ Min( x + 1, nCbSX − 1 ) ][ y ] ) >> 2

if( !(( tFlag | | cFlag | | bFlag ) && ( !tFlag | | !cFlag | | !bFlag) ) )

filt = ( p[ x ][ Max( 0, y − 1 ) ] + ( filt << 1 ) + p[ x ][ Min( y + 1, nCbSX − 1 ) ] ) >> 2

In further, because the condition in "if" can be simplified, I suggest it is changed to

if( lFlag != cFlag | | cFlag != rFlag )

filt = ( p[ Max( 0, x − 1 ) ][ y ] + ( filt << 1 ) + p[ Min( x + 1, nCbSX − 1 ) ][ y ] ) >> 2

if( tFlag != cFlag | | cFlag != bFlag )

filt = ( p[ x ][ Max( 0, y − 1 ) ] + ( filt << 1 ) + p[ x ][ Min( y + 1, nCbSX − 1 ) ] ) >> 2

Change history (3)

comment:1 Changed 10 years ago by DefaultCC Plugin

  • Cc tech jct-3v@… added

comment:2 Changed 10 years ago by tech

  • Version set to 3D-HEVC Draft Text 4

comment:3 Changed 10 years ago by tech

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.

This list contains all users that will be notified about changes made to this ticket.

These roles will be notified: Reporter, Owner, Subscriber, Participant

  • Gerhard Tech(Subscriber, Participant, Always)
  • jct-3v@…(Subscriber)
  • Karsten Suehring(Always)
  • Xianguo Zhang(Reporter)