Opened 12 years ago

Closed 12 years ago

#749 closed defect (fixed)

VPS encoder/decoder mismatch

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

Description (last modified by ksuehring)

Reading or writing of the VPS syntax elements seems to be broken (which does not have any effect on decoding because these syntax elements are currently ignored).

The trace file (#define ENC_DEC_TRACE 1) differs as follows:

encoder:

       0  video_parameter_set_id                   u(4) : 0
       1  vps_temporal_id_nesting_flag             u(1) : -1

decoder:

       0  video_parameter_set_id                   u(4) : 15
       1  vps_temporal_id_nesting_flag             u(1) : 1

Also a flag should never be "-1".

Attachments (1)

VPS_BUGFIX_749.patch​ (1.5 KB) - added by adarsh 12 years ago.
VPS_BUGFIX related to ticket #749

Download all attachments as: .zip

Change History (5)

comment:1 Changed 12 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf jct-vc@… added

comment:2 Changed 12 years ago by ksuehring

  • Description modified (diff)

comment:3 Changed 12 years ago by adarsh

The encoder writes the flag vps_temporal_id_nesting_flag with a "-1", which is a typo. This generated both the negative value and the wrong value for video_parameter_set_id. The attached patch (removing the -1) takes care of both the issues.

Changed 12 years ago by adarsh

VPS_BUGFIX related to ticket #749

comment:4 Changed 12 years ago by ksuehring

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

The patch has been committed in r2760

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

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