Opened 10 years ago

Closed 10 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 10 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf jct-vc@… added

comment:2 Changed 10 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)