Opened 12 years ago Closed 12 years ago #920 closed defect (fixed)Missing pocCRA update causes decoder assertion failure
Description
While testing HM-9.1 we discovered that the HM decoder encounters an assertion failure at line 600 in the TComSlice.cpp. The bitstream is valid however currently the value of variable pocCRA is not updated at the decoder when an IDR NALU type is encountered in the bitstream. The attached fix proposes a change that updates pocCRA when an IDR or IDR_N_LP NALU type is encountered. Attachments (3)Change History (10)comment:1 Changed 12 years ago by DefaultCC Plugin
Changed 12 years ago by kiranmisracomment:2 Changed 12 years ago by ksuehring
Sounds like a bad name for something that is not the POC of a CRA access unit. comment:3 Changed 12 years ago by fbossen
It would be helpful to have a corresponding bitstream to reproduce the error Changed 12 years ago by kiranmisracomment:4 Changed 12 years ago by kiranmisra
Attached is an example bitstream str.bin. It was created by concatenating a random access main bitstream with 50 frames and a low delay B main bitstream with 50 frames. The first bitstream has a pocCRA value of 32 which is never changed at the start of the second bitstreams IDR picture. As a result when within the second bitstream when POC 33 references pictures with POC lower than 32 it causes an assertion failure at the decoder. The proposed patch updates pocCRA when an IDR NALU type is encountered in the bitstream. comment:5 Changed 12 years ago by fbossen
Thank you for the stream. Can you clarify which version of HM was used to generate it? Changed 12 years ago by kiranmisracomment:6 Changed 12 years ago by kiranmisra
Apologies, it seems the earlier uploaded example bitstream was generated using an experimental version of the source (HM-9.1 with bug fixes 712, 827). I have regenerated the bitstream using HM-9.1 source and uploaded it "str_hm_91_pocCRA_assertion_example.bin". comment:7 Changed 12 years ago by fbossen
Fixed in r3196 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
|
Patch to update pocCRA for IDR