Some of sequence decode failed/mismatch to VTM-11.0

[chenm001]

Some of sequence decode failed/mismatch to VTM-11.0

[chenm001]

Sorry for delay, I only have time on weekends.
I think I found the root cause.
The bug in DecLibParser::checkNoOutputPriorPics , I explain details in here.
These sequence have small IDR period, so the decoder will meet a IDR before queue clear (especially my laptop have 8 threads, the decoder put 9 of delay frames in queue).
Moreover, the decoder is design to async parsing and decompress, so early call to DecLibParser::checkNoOutputPriorPics will be clear all of Decoded but No-output picture before IDR, this is reason that we lost some picture.
Are you want to me upload temporary fix?

[adamjw24]

If you have a temporary fix we would appreciate it. There is a plan to look more closely into the DPB management which would hopefully include those fixes, but can't yet say when will get to do that. So if you have a solution in the meantime, that would be great.

[chenm001]

I will submit pull request for my workaround, it pending in my self verify processing, my laptop a little slow.

btw:
In engineering, I likely to fix bugs use as simple as possible but stable method, to allow us avoid make new bug. In most China companies, this patch is not good because it only modify half-line, it means 'no productive code output' or equal to no KPI / salary over there. I hope open source fields does not so so.

[adamjw24]

Can this be closed after #21 integration?

[chenm001]

Yes, we can close this topic, thank you.

[adamjw24]

Yes, we can close this topic, thank you.

Thank you