View Issue Details

IDProjectCategoryView StatusLast Update
0000191JVT JM H.264/AVC reference softwareencoderpublic2011-05-19 12:21
ReporterPierre Andrivon Assigned ToKarsten Suehring  
PrioritynormalSeveritytweakReproducibilityalways
Status resolvedResolutionfixed 
Product VersionJM 16.1 
Target VersionJM 18.1Fixed in VersionJM 18.1 
Summary0000191: Bad stats regarding Intra coding (intraPredMode)
DescriptionAccording to JM16.1, it seems that IntraPredmode syntax elements (namely prev_intra4x4_pred_mode_flag and rem_intra_4x4_pre_mode) are added to the current mb_y_coeff stat element. This element stores post-RLE coeff stats (texture bits).
e.g. line 1755 in macroblock.c :

currSlice->writeIntraPredMode (&se, dataPart);

    rate += se.len;
  }
mbBits->mb_y_coeff += rate;

However, I think that this element rate should be instead added in mb_mode stat element as it should be considered like a header element.
Moreover, mb_mode already stores mb_type, transform_size_8x8_flag elements etc… that are MB header elements.
TagsNo tags attached.

Activities

Karsten Suehring

2011-05-19 12:19

administrator   ~0000478

fixed as suggested (for 4x4 and 8x8 writing)

Issue History

Date Modified Username Field Change
2009-10-21 09:33 Pierre Andrivon New Issue
2011-05-19 12:19 Karsten Suehring Note Added: 0000478
2011-05-19 12:19 Karsten Suehring Status new => resolved
2011-05-19 12:19 Karsten Suehring Fixed in Version => JM 18.1
2011-05-19 12:19 Karsten Suehring Resolution open => fixed
2011-05-19 12:19 Karsten Suehring Assigned To => Karsten Suehring
2011-05-19 12:21 Karsten Suehring Target Version => JM 18.1