Opened 9 years ago

Closed 9 years ago

#575 closed defect (fixed)

Resolution check is wrong when CroppingMode = 1

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

Description

When frame width or frame height is not a multiple of the minimum CU size, the frame width and hieight should be padded. When CroppingMode = 1, the HM should be able to pad the frame width and hieight automatically.
In TAppEncCfg.cpp line 418, a check is added to see if the number of padded pixels is a integer multiple of the specified chroma subsampling.
However, the
"if (m_aiPad[1] % TComSPS::getCropUnitY(CHROMA_420) != 1)"

should be

"if (m_aiPad[1] % TComSPS::getCropUnitY(CHROMA_420) != 0)"

Change History (2)

comment:1 Changed 9 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf jct-vc@… added

comment:2 Changed 9 years ago by fbossen

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

Fixed in r2413

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