NumView derivation seems not correct. The current code recognized 6 views when there is only 3 views. I found this bug when decoding the bitstream in ​http://wftp3.itu.int/av-arch/jct3v-site/bitstream_exchange/under_test/MV-HEVC/MVHEVCS_I_Nokia_2.zip (but some additional workaround is needed for testing with the bistream)

The current code in getNumViews() [TComSlice.cpp]

Int lId = m_layerIdInNuh[i]; if( i > 0 && ( getViewIndex( lId ) != getScalabilityId( i - 1, VIEW_ORDER_INDEX ) ) ) {

numViews++;

}

should be replaced with

Int lId = m_layerIdInNuh[i]; if ( i > 0 ) {

Bool newViewFlag = true; for( Int j = 0; j < i; j++ ) {

if( getViewIndex( lId ) == getScalabilityId( getLayerIdInNuh(j), VIEW_ORDER_INDEX ) ) {

newViewFlag = false;

}

} if( newViewFlag ) {

numViews++;

}

}

When NumLayerSets is equal to 2, layer_set_idx_for_ols_minus1 is not present. Thus it should be inferred to be 0 rather than i in the SHM. The verion 2 spec reads: "When not present, the value of layer_set_idx_for_ols_minus1[ i ] is inferred to be equal to 0".

A patch is attached (LAYER_SET_IDX_FOR_OLS_BUGFIX). For testing, we may use the bitstream in ​http://wftp3.itu.int/av-arch/jct3v-site/bitstream_exchange/under_test/MV-HEVC/MVHEVCS_A_Qualcomm_2.zip.

From software version SHM 5.1, the LayerConfig configuration option is not fully implemented; it is not active. For example, when using the command switch "-lc0" to specify a specific configuration file for the base layer, this command is ignored during the encoding process and no error is produced. The same is true for all layers. I have included a screenshot image (below) of the configuration option to which I am referring.

In the TAppEncCfg.cpp source code file, the string variable "cfg_LayerCfgFile" in the public member function "parseCfg" of class "TAppEncCfg" (starting at line 338) is relevant to this issue.