Opened 13 years ago

Closed 11 years ago

#157 closed defect (fixed)

FIXED_ROUNDING_FRAME_MEMORY modifies output picture rather than reference picture

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

Description (last modified by davidf)

In TEncGOP.cpp:

      xCalculateAddPSNR( pcPic, pcPic->getPicYuvRec(), numBits, dEncTime );


xFixedRoundingPic() causes both the output picture and any reference picture to be modified.

xFixedRoundingPic() in the encoder should be performed after pcPic->getPicYuvRec()->copyToPic(pcPicYuvRecOut);. In the decoder, a similar change is required, however, the decoder does not currently have a separate reorder buffer.

Change History (6)

comment:1 Changed 13 years ago by fbossen

My understanding is that this should modify only the picture stored in the reference buffer.

comment:2 Changed 13 years ago by davidf

  • Description modified (diff)
  • Milestone changed from HM-3.1 to HM-3.2
  • Summary changed from FIXED_ROUNDING_FRAME_MEMORY modifies output picture after PSNR is calculated to FIXED_ROUNDING_FRAME_MEMORY modifies output picture rather than reference picture

Deferring until later. The fix for the encoder is trivial, however, the decoder fix isn't.

It should be noted that the reported data for the HM-3.0 software with FIXED_ROUNDING_FRAME_MEMORY enabled is correct, however, the output yuv files are not.

comment:3 Changed 12 years ago by davidf

  • Component set to HM

Updating component after adding WD (Text) tickets

comment:4 Changed 12 years ago by davidf

  • Cc fbossen ksuehring davidf added

comment:5 Changed 12 years ago by davidf

  • Cc jct-vc@… added

comment:6 Changed 11 years ago by davidf

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

FIXED_ROUNDING_FRAME_MEMORY has been removed in r2921 for HM-9.0. The issue can no longer occur.

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