Opened 9 years ago

Closed 9 years ago

#460 closed defect (fixed)

text mismatch in check wether log2TrafoSize == Log2MinTrafoSize

Reported by: bbross Owned by:
Priority: minor Milestone:
Component: HM Version: HM-6.1
Keywords: Cc: fbossen, ksuehring, davidf, jct-vc@…

Description

The TU processing and the chroma cbf parsing in the draft is conditioned on log2TrafoSize==Log2MinTrafoSize whereas in HM it is conditioned on log2TrafoSize==2, as in e.g. DecEntropy.cpp l.623 (DecEntropy::xDecodeTransform):

if( uiLog2TrafoSize > 2 )

...

else

...
Thus, instances where a log transform size is compared to 2 should be replaced with a comparison to the min log transform size.

Furthermore to ensure that this is working the bitstream shall not contain data that result in Log2MinTrafoSize >= Log2MinCbSize. (see http://hevc.kw.bbc.co.uk/trac/ticket/459)
This is taken care of by the encoder cfg but it should be considered at the decoder as well.

Note that this does not affect the common test conditions.

Change History (2)

comment:1 Changed 9 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf jct-vc@… added

comment:2 Changed 9 years ago by bbross

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

Due to JCTVC-G112 adoption, HM is right is CD text should be corrected. Text fix can be found in JCTVC-I0030 (v1).

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(Reporter, Participant)
  • David Flynn(Subscriber)
  • Frank Bossen(Subscriber)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Subscriber, Always)