Opened 9 years ago

Closed 9 years ago

#520 closed defect (fixed)

Derivation of iNumRowTilesMinus1 appears to be wrong (TDecCAVLC.cpp)

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

Description

In HM-6.1, on line 1269 of TDecCAVLC.cpp, the derivation of iNumRowTilesMinus1 appears to be wrong.

Int iNumRowTilesMinus1 = (pcPPS->getColumnRowInfoPresent() == 1)?(pcPPS->getNumColumnsMinus1()):(pcPPS->getSPS()->getNumRowsMinus1());

Is it correct to derive iNumRowTilesMinus1 from pcPPS->getNumColumnsMinus1()?

Change History (3)

comment:1 Changed 9 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf jct-vc@… added

comment:2 Changed 9 years ago by ksuehring

  • Owner set to ksuehring
  • Status changed from new to assigned

This looks like a copy and paste bug. The same issue is there in the encoder.

The code takes care of the PPS override process of the SPS parameters which will be removed after that meeting.

comment:3 Changed 9 years ago by ksuehring

  • Milestone set to HM-6.3
  • Resolution set to fixed
  • Status changed from assigned to closed

fixed in r2318

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)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Owner, Subscriber, Participant, Always)
  • Sandeep Kanumuri(Reporter)