Opened 8 years ago

Closed 7 years ago

#841 closed defect (fixed)

Monochrome support - syntax table changes

Reported by: crosewarne Owned by: davidf
Priority: major Milestone: RExt D7
Component: RExt text Version: D10 (L1003) v33
Keywords: Cc: bbross, wjhan, jct-vc@…

Description

This ticket captures syntax table (and consequent) changes necessary to support monochrome (4:0:0).

In K1003-v9:

7.3.2.2 Sequence parameter set RBSP syntax:

...
bit_depth_luma_minus8
if( chroma_format_idc != 0 ) <= New condition

bit_depth_chroma_minus8

...
pcm_sample_bit_depth_luma_minus1
if( chroma_format_idc != 0 ) <= New condition

pcm_sample_bit_depth_chroma_minus1

...

7.3.2.3 Picture parameter set RBSP syntax:

...
if( chroma_format_idc != 0 ) { <= New condition

pic_cb_qp_offset
pic_cr_qp_offset
pic_slice_level_chroma_qp_offets_present_flag

} <= New
...

7.3.6 Scaling list data syntax:

...
for( sizeId = 0; sizeId < 4; sizeId++ )

for( matrixId = 0; matrixId < ( sizeId == 3 ) ? 2 : 6; matrixId++ ) {

...
Replace with:
...
for( sizeId = 0; sizeId < 4; sizeId++ ) {

totalMatrices = ( ( chroma_format_idc == 0 )
(sizeId == 3 ) ) ? 2 : 6

for( matrixId = 0; matrixId < totalMatrices; matrixId++ ) {

...
}
...

7.3.8.1 General slice segment header syntax:

...
slice_sao_luma_flag
if( ChromaArrayType != 0 ) <= New condition

slice_sao_chroma_flag

...

7.3.9.3 Sample adaptive offset syntax:

...
for( cIdx = 0; cIdx < 3; cIdx++ ) {
...
Replace with:
...
for( cIdx = 0; cIdx < ( ( ChromaArrayType == 0 ) ? 1 : 3 ); cIdx++ ) {
...

7.3.9.5 Coding unit syntax:

...
if( ChromaArrayType != 0 ) <= New condition

intra_chroma_pred_mode[ x0 ][ y0 ]

...

7.3.9.7 PCM sample syntax:

...

pcm_sample_luma[i]

for( i = 0; i < ( 1 < ( log2CbSize << 1 ) ) >> 1; i++ )

pcm_sample_chroma[i]

...
Replace with:
...

pcm_sample_luma[i]

if( ChromaArrayType != 0 )

for( i = 0; i < ( ( ( 1 << log2CbSize ) / ( SubWidthC * SubHeightC ) ) << 1); i++ )

pcm_sample_chroma[i]

...

7.3.9.8 Transform tree syntax:

...
if( log2TrafoSize > 2 ) {
...
Replace with:
...
if( log2TrafoSize > 2 && ChromaArrayType != 0 ) {
...

7.4.2.3 Picture parameter set RBSP semantics:

...
pic_slice_chroma_qp_offsets_present_flag equal to 1 indicates that the slice_cb_qp_offset and slice_cr_qp_offset syntax elements are present in the associated slice headers. pic_slice_chroma_qp_offsets_present_flag equal to 0 indicates that these syntax elements are not present in the associated slice headers.
...
Replace with:
...
pic_slice_chroma_qp_offsets_present_flag equal to 1 indicates that the slice_cb_qp_offset and slice_cr_qp_offset syntax elements are present in the associated slice headers. pic_slice_chroma_qp_offsets_present_flag equal to 0 indicates that these syntax elements are not present in the associated slice headers. When pic_slice_chroma_qp_offsets_present_flag is not present, it is inferred to be equal to 0.
...

Change History (15)

comment:1 Changed 8 years ago by DefaultCC Plugin

  • Cc bbross wjhan jct-vc@… added

comment:2 Changed 8 years ago by kazushi

Pls also refer to JCTVC-I0037.

comment:3 Changed 8 years ago by bbross

  • Version changed from D9 (K1003) v9 to D9 (K1003) v10

comment:4 Changed 8 years ago by bbross

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

comment:5 Changed 8 years ago by bbross

  • Resolution fixed deleted
  • Status changed from closed to new

comment:6 Changed 8 years ago by bbross

  • Version changed from D9 (K1003) v10 to D9 (K1003) v11

comment:7 Changed 8 years ago by bbross

  • Version changed from D9 (K1003) v11 to D10 (L1003) v1

comment:8 Changed 8 years ago by bbross

  • Milestone set to D10

comment:9 Changed 8 years ago by bbross

  • Version changed from D10 (L1003) v1 to D10 (L1003) v2

comment:10 Changed 8 years ago by bbross

  • Version changed from D10 (L1003) v2 to D10 (L1003) v28

comment:11 Changed 8 years ago by bbross

  • Milestone D10 deleted

comment:12 Changed 8 years ago by bbross

  • Version changed from D10 (L1003) v28 to D10 (L1003) v31

comment:13 Changed 8 years ago by bbross

  • Version changed from D10 (L1003) v31 to D10 (L1003) v33

comment:14 Changed 7 years ago by marcin.jasinski

  • Component changed from Text to RExt text
  • Owner changed from bbross to davidf
  • Priority changed from minor to major
  • Type changed from enhancement to defect
  • Version changed from D10 (L1003) v33 to RExt D7 (Q1005) v8

SPS and PPS still not fixed in Q1005 v8

comment:15 Changed 7 years ago by davidf

  • Milestone set to RExt D7
  • Resolution set to fixed
  • Status changed from new to closed
  • Version changed from RExt D7 (Q1005) v8 to D10 (L1003) v33

The following were modified during the course of RExt development:

  • Scaling lists
  • SAO (slice header / sao syntax)
  • Coding unit
  • PCM syntax
  • Transform tree syntax
  • Semantics of pic_slice_chroma_qp_offsets_present_flag

No changes were made to the existing SPS and PPS other than the addition of extensions using the extension mechanism in version 1. The SPS and PPS have been designed such that there is no requirement to interpret the profile in order to parse them.

NB, the changes in RExt are not necessarily the same as what is suggested in this ticket.

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

  • Benjamin Bross(Subscriber, Participant)
  • David Flynn(Owner, Participant)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Always)
  • Kazushi Sato(Participant)
  • Marcin Jasiński(Participant)
  • Woo-Jin Han(Subscriber)