Opened 7 years ago

Closed 7 years ago

#1108 closed defect (fixed)

No implementation of DPB dumping when encountering a new CVS starting with a CRA picture

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

Description

The spec, section C.3.2, says:

If the current picture is a CRA picture [with NoRaslOutputFlag equal to 1], NoOutputOfPriorPicsFlag is set equal to 1 (regardless of the value of no_output_of_prior_pics_flag).

So when a new CVS is encountered whose first picture is a CRA picture, the tail end of the previous CVS is dumped without output. The reference decoder doesn't appear to implement this rule. I'm also somewhat confused as to why this rule exists - why particularly would you want to dump the contents of the DPB in this situation?

Change History (4)

comment:1 Changed 7 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf jct-vc@… added

comment:2 Changed 7 years ago by ksuehring

A new CVS is only started when the CRA picture has NoRaslOutputFlag equal to 1. This can only happen, when HandleCraAsBlaFlag is equal to one. We don't have an implementation that sets HandleCraAsBlaFlag.

Thus I don't think, we need to implement setting of NoOutputOfPriorPicsFlag based on that condition.

comment:3 Changed 7 years ago by ksuehring

Ah, I see: NoRaslOutputFlag can is also set for any picture that allows an EOS_NUT. That means, the fix in issue #1101 needs to be modified to discard prior pics instead of outputting them.

comment:4 Changed 7 years ago by fbossen

  • Milestone set to HM-14.0
  • Resolution set to fixed
  • Status changed from new to closed

Fixed in r3852

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

  • David Flynn(Subscriber)
  • Frank Bossen(Subscriber, Participant)
  • Jack Haughton(Reporter)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Subscriber, Participant, Always)