View Issue Details

IDProjectCategoryView StatusLast Update
0000086JVT JM H.264/AVC reference softwareencoderpublic2007-12-17 12:31
ReporterLuca Assigned ToKarsten Suehring  
PrioritynormalSeverityminorReproducibilityalways
Status resolvedResolutionfixed 
Product VersionJM 13.1 
Fixed in VersionJM 13.2 
Summary0000086: Outputted average QP with slice dimension fixed in bytes
DescriptionDear Experts,

I faced a problem with the output on-screen when encoding sequences with slicing mode "2" (slice dimension fixed in bytes).

The quantization parameter indicated, usually was bigger than the one I selected (without RDO of course). I found out that it is calculated after completing the encoding as

img->AverageFrameQP = (img->SumFrameQP + (img->FrameSizeInMbs >> 1))/img->FrameSizeInMbs;

Actually I found out that the variable img->SumFrameQP is updated in write_one_macroblock as

img->SumFrameQP += currMB->qp;

each time a MB is encoded. Keeping a fixed slice dimension, a MB can be encoded twice: a first time in case it, as last MB of a slice, let the current slice exceed the threshold, and a second time as first MB of the following slice.

Therefore, to my understanding, the img->SumFrameQP should not be updated in case the "recode_macroblock" flag is set to "TRUE", and possibly, also the trace file should not be updated.

Since this is my first report, please tell me if the reported information were sufficient, and please tell me also if I can help you solving this.
TagsNo tags attached.

Activities

Karsten Suehring

2007-12-17 12:31

administrator   ~0000135

A fix from Thanasis has been integrated into the development tree.

Issue History

Date Modified Username Field Change
2007-12-12 19:46 Luca New Issue
2007-12-12 21:07 Karsten Suehring Category decoder => encoder
2007-12-17 12:31 Karsten Suehring Status new => resolved
2007-12-17 12:31 Karsten Suehring Fixed in Version => JM 13.1-dev
2007-12-17 12:31 Karsten Suehring Resolution open => fixed
2007-12-17 12:31 Karsten Suehring Assigned To => Karsten Suehring
2007-12-17 12:31 Karsten Suehring Note Added: 0000135