Opened 10 years ago

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

  • Cc fbossen ksuehring davidf jct-vc@… added

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