There seems to be a discrepancy between spec and software. The spec reads that bipred prevention is applied regardless of VSP flag:

14. When predFlagL0 is equal to 1 and predFlagL1 is equal to 1, and ( nOrigPbW + nOrigPbH ) is equal to 12, the following applies:

refIdxL1 = −1 (I 112) predFlagL1 = 0 (I 113)

However, the software (in TDecEntropy.cpp and TEncSearch.cpp)disbles it in the case VSP flag is true as follows. #if H_3D_VSP

if ( (pcCU->getInterDir(uiSubPartIdx) == 3) && pcSubCU->isBipredRestriction(uiPartIdx) && (pcCU\

->getVSPFlag(uiSubPartIdx) == 0)) #else

if ( (pcCU->getInterDir(uiSubPartIdx) == 3) && pcSubCU->isBipredRestriction(uiPartIdx) )

#endif

We should remove it (spec should be right).

In CTC and IBP(with MTK_I0072_IVARP_SCALING_FIX) experiment, there are no differences by this fix. No change is quite natural because VSP PU cannot be bi-prediction in the current specification. If looking at predFlagLXVSP setting logic(See I.8.5.3.2.13 Derivation process for a view synthesis prediction merge candidate), when predFlagL0VSP is 1, predFlagL1VSP cannot be 1. When predFlagL0VSP is 0, the PU is not bi-prediction. The same logic is applied in VSP inhericance case.

Lastly, even if VSP was able to be bi-prediction, we should prevent 8x4/4x8 bipred in VSP as well to preserve motion data storing implemenation.

Currently view dependent options from configuration file are parsed wrong. In particular more than 10 views cannot be parsed correctly. Following bugfix solves the problem

In file program_options_lite.h,line 389 following should be added cNameBuffer .resize( name.size() + 10 );

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