Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MediaInfo is not returning the ChannelPositions field for MXF file #1654

Open
paul-aloisio opened this issue Jan 9, 2023 · 3 comments
Open
Assignees
Labels

Comments

@paul-aloisio
Copy link

I discovered an issue with MediaInfo not returning some metadata for an MXF file in a newer version of FITS ( toolname="MediaInfo" toolversion="22.09"), vs. FITS 1.5.5. (toolname="MediaInfo" toolversion="0.7.75")

The 30MB file I used is in this share drive:
https://drive.google.com/drive/folders/1kWy6yRxyDLhkVtSbFwTBPZa9TgPPnV63?usp=share_link
MXF_44_1_1_fits.zip

The issue was reported in the FITS project here:
harvard-lts/fits#369
Thanks
Paul

@JeromeMartinez JeromeMartinez self-assigned this Jan 9, 2023
@JeromeMartinez
Copy link
Member

There is a regression for sure as I see well the channel info in the file.

But files says that channel layout is EBU R48 "2a", so stereo (L+R) and D-10 audio stream says that 4 channels are active, so it is incoherent. What is the truth? As it is SD content, I would bet on stereo and I think that I'll show something like "L R ? ?" instead of previous "L C R, LFE" (I am a bit lazy and don't plan to look for the source of this info in the past).

@paul-aloisio
Copy link
Author

hi @JeromeMartinez - did you need some information from me? My main concern is getting the metadata that had been available before.

@JeromeMartinez
Copy link
Member

My main concern is getting the metadata that had been available before.

I checked with (the very old, 7 years) 0.7.75 version, and "L C R, LFE" comes from the idea that a 4-channel is by default "L C R, LFE" but in reality SMPTE ST 365 (D-10 Audio) and SMPTE ST 331 (the SDTI storage format used in this MXF) indicate the count of channels but no default channel mapping, so we removed this output on purpose and it won't be available again because it is wrong.

But in the case of your file, we see another piece of metadata, from AS-11 UK DPP (which was not supported in 0.7.75) so we plan to add "L R" (because AS-11 UK DPP metadata in this file says that) even if it is a 4-ch track.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants