View Issue Details

IDProjectCategoryView StatusLast Update
0000340JVT JM H.264/AVC reference softwaredecoderpublic2015-07-08 16:27
Reporterashkan asgharzade Assigned ToKarsten Suehring  
PriorityhighSeveritymajorReproducibilityhave not tried
Status acknowledgedResolutionopen 
OSwin 7 OS Version64bit 
Product VersionJM 18.6 
Summary0000340: error in motion vector copy concealment at the decoder
DescriptionHi, there
 we encode 10 frames in baseline profile and it gives us 12 RTP packets.
 after that we run rtp_loss with 18% PLR the result is that the packet number 10 is lost that makes frame number 9 is lost. when we run ldecod.exe with frame copy concealment there isn't any problem but when we run it with motion vector copy then decoder can't decode video and rec file is then 0Kb. when we traced the ldecod.exe we found that when decoder reaches to frame number 9 that is lost and wants to conceal it in the function copy_to_conceal the value of p_Vid->dec_picture is null. in frame copy we don't have any problem (because there is no need to this parameter) but in motion vector copy because it uses this parameter in function buildPredblockRegionYUV then it can't continue it's concealing process. we want to ask that whether this issue is logical and we should repair it or not? thank's a lot if you help us to solve this issue.


Thanks A lot
TagsNo tags attached.

Activities

Karsten Suehring

2014-04-28 17:16

administrator   ~0000604

There are several reports on problems in error concealment in this bug tracker. The code has not been maintained well over the last years. Any help with fixing is appreciated.

ashkan asgharzade

2014-05-02 12:45

reporter   ~0000609

Thanks a lot because of your answer. I have another question. my question is that do you have any reports specifically about error in motion vector copy concealment?

Thanks a lot

Karsten Suehring

2014-05-05 16:41

administrator   ~0000610

for instance issue 0000004

João Carreira

2014-06-13 19:49

reporter   ~0000611

Dear all,

I did a master dissertation in error concealment for 3D video, so I am well aware of the decoder problems in terms of concealment.

I am available to provide some code that could be used to fix the problems in both 2D and stereo video.

Karsten Suehring

2014-08-07 14:57

administrator   ~0000616

Please attach any patches that you have to this bug report.

Thanks.

João Carreira

2015-07-07 15:43

reporter   ~0000635

Is still required concealment functions into JM H.264?

What methods are needed? frame-copy and motion-copy?

What are the code style required in order to the patch to be applied?

I was working in a different subject last year and I still need to make some updates to my code in order to match the current version

Karsten Suehring

2015-07-08 15:32

administrator   ~0000637

Frame and motion copy are currently in the software. It seems frame copy is still working to some extend, but motion copy is completely broken.

It would be nice, if we can fix this. Bit if it stays broken, the best alternative is to completely remove all error concealment code.

There is only a rather minimal coding style, see the file "coding_style.doc" in the doc directory or

http://wftp3.itu.int/av-arch/video-site/0109_San/VCEG-N46.doc

João Carreira

2015-07-08 16:27

reporter   ~0000638

In terms of code style there might not be any problems.

I have ported my changes into the version 19.0 from the 16.2.

MY approach takes the original function for frame loss concealment of erc_do_p.c and redesigned it maintaining some ideas and improving it especially to detect the frame loss and detect what is the lost frame (e.g., frame POC). However, it involves using new functions names and adding some functions.

Would it be a problem?

Issue History

Date Modified Username Field Change
2014-04-28 12:32 ashkan asgharzade New Issue
2014-04-28 17:16 Karsten Suehring Note Added: 0000604
2014-05-02 12:45 ashkan asgharzade Note Added: 0000609
2014-05-05 16:41 Karsten Suehring Note Added: 0000610
2014-06-13 19:49 João Carreira Note Added: 0000611
2014-08-07 14:57 Karsten Suehring Note Added: 0000616
2014-08-07 14:57 Karsten Suehring Assigned To => Karsten Suehring
2014-08-07 14:57 Karsten Suehring Status new => acknowledged
2015-07-07 15:43 João Carreira Note Added: 0000635
2015-07-08 15:32 Karsten Suehring Note Added: 0000637
2015-07-08 16:27 João Carreira Note Added: 0000638