Opened 9 years ago

Closed 9 years ago

#919 closed defect (fixed)

Chroma QP offsets not use in forward quantization

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

Description

When RDOQ is off, the forward quantization does not use chroma QP offsets. Dequantization correctly uses the Chroma QP offsets during reconstruction. The encoder and decoder are matched but the quantization process may introduce unintended error. For example reducing QP by using a value of -12 for CrQpOffset should increase the V psnr but the psnr is lowered compared to CrQpOffset of zero.

A patch is attached.

Attachments (1)

TComTrQuant.cpp (87.3 KB) - added by lkerofsky 9 years ago.
patch to xQuant() for ChromaQPOffsets

Download all attachments as: .zip

Change History (3)

comment:1 Changed 9 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf jct-vc@… added

Changed 9 years ago by lkerofsky

patch to xQuant() for ChromaQPOffsets

comment:2 Changed 9 years ago by fbossen

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

Fixed in r3163
BTW a .cpp file is not a patch file. The following may be useful to some Windows users:
http://tortoisesvn.net/docs/release/TortoiseSVN_en/tsvn-dug-patch.html

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)
  • Louis Kerofsky(Reporter)