Opened 9 years ago

Closed 9 years ago

#547 closed defect (fixed)

Periodic I frames in Low Delay Configuration

Reported by: basakoztas Owned by:
Priority: minor Milestone:
Component: HM Version: HM-7.0
Keywords: low delay, configuration, periodic I Cc: fbossen, ksuehring, davidf, jct-vc@…

Description

In the default low delay configuration file within the HM7.0, when the I frame period is set to a positive value (let's say 8) and decoding refresh type is set to zero (I frame) to create a sequence with periodic I frames, decoder crashes with the following error:

Assertion failed: getPOC()+pReferencePictureSet->getDeltaPOC(i) >= pocCRA, file <path>\source\lib\tlibcommon\tcomslice.cpp, line 542

Attachments (1)

547_rickard.patch (731 bytes) - added by rickard 9 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 9 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf jct-vc@… added

comment:2 Changed 9 years ago by basakoztas

Apperantly, reference Picture Sets are explicitely specified in the parameter sets to prevent problems in the references for the first couple of frames (as they don't have some of the reference pictures yet in the beginning of a sequence). However, when a periodic I frame insertion is dictated in the configuration, the encoder doesn't check if all the reference pictures are available and default value is given. In this case, for example for I frame with POC 8, it tries to use a non-existent frame which is 8 - (12) = -4

comment:3 Changed 9 years ago by rickard

The attached patch "547_rickard.patch" provides a fix to the problem.

Changed 9 years ago by rickard

comment:4 Changed 9 years ago by fbossen

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

Fixed in r2463

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

  • Basak Oztas Yoldemir(Reporter, Participant)
  • David Flynn(Subscriber)
  • Frank Bossen(Subscriber, Participant)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Subscriber, Always)
  • Rickard Sjoberg(Participant)