SEncoderStatistics.uiAverageFrameQP always return 0 when get the encoder stats

[mypopydev]

SEncoderStatistics.uiAverageFrameQP always return 0 when get the encoder stats with option ENCODER_OPTION_GET_STATISTICS

[huili2]

any detailed case for this issue? Actually If the encoder is called properly, and encoding is just in the process, the return value should be true.

[mypopydev]

I think uiAverageFrameQP means Average Frame QP base on Frame, but openh264enc just return 0 now

[huili2]

your guess is right. But it is strange that it returns 0. Any more details on how it occurs?

[mypopydev]

You can try FFmpeg + Openh264enc with the patch mypopydev/FFmpeg@9f95625

[Allen1207]

that because the uiAverageFrameQP is not assigned when GetOption ENCODER_OPTION_GET_STATISTICS. You may add one line "pStatistics->uiAverageFrameQP = pEncStatistics->uiAverageFrameQP;" as follows to solve it:

case ENCODER_OPTION_GET_STATISTICS: {
SEncoderStatistics* pStatistics = (static_cast<SEncoderStatistics*> (pOption));
SEncoderStatistics* pEncStatistics = &m_pEncContext->sEncoderStatistics[m_pEncContext->pSvcParam->iSpatialLayerNum - 1];
pStatistics->uiWidth = pEncStatistics->uiWidth;
pStatistics->uiHeight = pEncStatistics->uiHeight;
pStatistics->fAverageFrameSpeedInMs = pEncStatistics->fAverageFrameSpeedInMs;
pStatistics->sEncMv = pEncStatistics->sEncMv;

// rate control related
pStatistics->fAverageFrameRate = pEncStatistics->fAverageFrameRate;
pStatistics->fLatestFrameRate = pEncStatistics->fLatestFrameRate;
pStatistics->uiBitRate = pEncStatistics->uiBitRate;
pStatistics->uiAverageFrameQP = pEncStatistics->uiAverageFrameQP;    //added line here

pStatistics->uiInputFrameCount = pEncStatistics->uiInputFrameCount;
pStatistics->uiSkippedFrameCount = pEncStatistics->uiSkippedFrameCount;

pStatistics->uiResolutionChangeTimes = pEncStatistics->uiResolutionChangeTimes;
pStatistics->uiIDRReqNum = pEncStatistics->uiIDRReqNum;
pStatistics->uiIDRSentNum = pEncStatistics->uiIDRSentNum;
pStatistics->uiLTRSentNum = pEncStatistics->uiLTRSentNum;

}

[mypopydev]

Will try this patch, thx

[mypopydev]

@huili2 can you try Allen1207's patch for this issue

[huili2]

I do not have code at hand, but from above code lines, I think it should be OK.