Issues on chroma BDPCM signaling

[geonjungko]

This ticket reports two issues on the chroma BDPCM signaling.

1.
In the current spec, if intra_bdpcm_chroma_flag is parsed and is equal to 0, other chroma intra mode syntax elements are not parsed.

Suggested fix:

if( conditions ) {

intra_bdpcm_chroma_flag

}

if( intra_bdpcm_chroma_flag)

intra_bdpcm_chroma_dir_flag

} else {
}

2.
The chroma BDPCM signaling exists in different places for the spec and the software.
Chroma BDPCM is signaled after the luma intra mode signaling in the spec, but it is signaled right after the luma BDPCM signaling before other luma intra mode signalings in the software.
It is not sure which is correct.

[gordon]

A proponent here -- for that second point it is the specification that is correct.

We will open a code ticket to bring the code into line with the specification, along with a merge request.

[jonathang]

I am crosschecking contribution JVET-Q0351, which appears to address aspect 2 of this ticket, as well as ticket #708. In both cases Q0351 is modifying the software to match the spec.

[bbross]

Thanks for reporting!

This has been solve by SW adoption at the last meeting:
Decision(CTC+BF/SW): Adopt JVET-Q0785 (the SW includes bug fixes JVET-Q0110 and JVET-Q0351)