H264 long gop is not supported yet?

[syehoonkim]

Hello,
I am trying to remux H264 lgop mxf file by bmxtranswrap, but the output goes wrong.
The output cannot be imported by Adobe Premiere, and when I try to play it and seek by mpv, it says like below:

Invalid video timestamp: 11.678333 -> 11.644966
Invalid video timestamp: 11.644966 -> 11.611600
[ffmpeg/video] h264: Missing reference picture, default is 2147483647
[ffmpeg/video] h264: Missing reference picture, default is 2147483647
[ffmpeg/video] h264: Missing reference picture, default is 2147483647
[ffmpeg/video] h264: Missing reference picture, default is 2147483647
[ffmpeg/video] h264: reference picture missing during reorder

I assume H264 lgop is not supported perfectly yet, and I wonder if you have any plan to do it.
Thank you.

[philipnbbc]

bmx has support for lgop but I'm sure it has limitations. I can't tell from those messages what the issue is but suspect it's how the content has been indexed. Are you able to provide the input file so I could see what the issue could be? Alternatively, provide the output of MXFDump --no-limit-entries <mxf file> for the input and bmxtranswrap output?

[syehoonkim]

Dear,
You can get the input file at below URL:
https://drive.google.com/file/d/1dN1dyxHv-e0c65ThgLljO2hHvluB9yOM/view?usp=drive_link
And I attach the mxfdump output file.
mxfdump_output.txt

Below is bmxtranswrap output of command bmxtranswrap -o output.mxf --clip test Clip0003.mxf:

Warning (libMXF): Failed to de-reference strong reference array element (item key = 06 0e 2b 34 01 01 01 09 06 01 01 04 06 10 00 00)
Warning: Not passing through ANC data track 5
Warning: Ignoring track 5 (essence type 'ANC data')
Warning: AVC support is work-in-progress
Info: Duration: 945 (00:00:31:15 @30Hz)

Thank you.

[philipnbbc]

Thanks for the file and info. I've had a look and found that the H.264 video uses separate field encoding and that's not supported in bmx. I'll see when I have time to figure out how far off support for separate fields was as it was years ago since I worked on this.

[syehoonkim]

Thank you!!

[philipnbbc]

It turns out it was fairly straightforward to add support for separate field encoded AVC. PR #94 has been created for this.

@syehoonkim would it be ok if I used a GOP of the AVC from the sample file you provided to use in a regression test?

[syehoonkim]

Dear,
Please do that.
I appreciate!!

[philipnbbc]

Thanks! I've made some changes to the PR and added a regression test.

[syehoonkim]

Dear,
For now, it seems working good.
Hope it be merged soon!!

[philipnbbc]

Fixed by #94