Opened 8 years ago

Closed 6 years ago

#1451 closed technical change (fixed)

not clear spelling

Reported by: kolya Owned by:
Priority: minor Milestone:
Component: Text Version:
Keywords: Cc: bbross, wjhan, jct-vc@…

Description

In the spec, there is:

pcm_enabled_flag equal to 0 specifies that PCM data are not present in the CVS.
NOTE 4 – When MinCbLog2SizeY is equal to 6, PCM data are not present in the CVS even when pcm_enabled_flag is equal to 1. The maximum size of coding block with pcm_enabled_flag equal to 1 is restricted to be less than or equal to Min( CtbLog2SizeY, 5 ). Encoders are encouraged to use an appropriate combination of log2_min_luma_coding_block_size_minus3, log2_min_pcm_luma_coding_block_size_minus3, and log2_diff_max_min_pcm_luma_coding_block_size values when sending PCM data in the CVS.

It is not clear what PCM data are assumed not to be in the stream: only PCM samples, or related syntax like pcm_sample_bit_depth_luma_minus1 etc.

Change History (4)

comment:1 Changed 8 years ago by kolya

The spec seems to be ill-formed: when log2_min_luma_coding_block_size_minus3 is equal to 3; log2_min_pcm_luma_coding_block_size_minus3 + 3 is expected to vary in the range [6,5], i.e., from higher to lower value.

At the same time, according to NOTE 4, the spec allows pcm_enabled_flag to be equal to 1, and log2_min_pcm_luma_coding_block_size_minus3 equal to 2 is allowed to present in the stream.

It seems the spec could be fixed by forcing pcm_enabled_flag to be set to zero with log2_min_luma_coding_block_size_minus3 equal to 3; instead of current edition of the NOTE 4.

Version 0, edited 8 years ago by kolya (next)

comment:2 Changed 8 years ago by kolya

  • Type changed from technical change to defect

comment:3 Changed 8 years ago by kolya

  • Type changed from defect to technical change

comment:4 Changed 6 years ago by bbross

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

This issue is fixed in the 2016/12 release of the HEVC specification.

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)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Always)
  • Nikolay Shlyakhov(Reporter, Participant)
  • Woo-Jin Han(Subscriber)