-
Notifications
You must be signed in to change notification settings - Fork 11
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
codec info bugs and missing data #2024
Comments
Comment #1 originally posted by Omertron on 2011-07-03T07:01:36.000Z: Forgot to add: you can find a good reasonable sized test package to run mediainfo against at: http://community.mediabrowser.tv/permalinks/3293/mediatype-sample-videos-set Some fields do not appear when they aren't part of the codec. |
Comment #2 originally posted by Omertron on 2011-11-26T08:09:39.000Z: <empty> |
Comment #3 originally posted by Omertron on 2011-11-27T07:51:30.000Z: Issue 2187 has been merged into this issue. |
Comment #4 originally posted by Omertron on 2011-11-27T07:52:25.000Z: Issue 2174 has been merged into this issue. |
Comment #5 originally posted by Omertron on 2011-11-28T18:56:06.000Z: The new format will be: The old format will be kept for backward compatibility. |
Comment #6 originally posted by Omertron on 2011-11-28T20:44:02.000Z: This issue was updated by revision r2819. MediaInfo codec information added to new codecs node |
Comment #7 originally posted by Omertron on 2011-11-28T21:03:31.000Z: Will this have any impact on the overlay codec? |
Comment #8 originally posted by Omertron on 2011-11-28T21:09:32.000Z: Hopefully, no. However, it would be nice for overlay to use the new values so that they are more accurate |
Comment #9 originally posted by Omertron on 2011-12-04T22:05:39.000Z: This issue was closed by revision r2824. |
Original issue 2025 created by Omertron on 2011-07-03T07:00:32.000Z:
Mediainfo needs to be completely revisited. The way data is saved is not a full picture of the format being used. In some case it could mean any of a dozen different variations.. codec_id is not enough..
Examples:
This is one of those variations that would probably break existing so many 2 new codec fields with more info might be better to add.
From what I've pieced together, these are the fields that matter to figure things out. If they were predictable like field1|field2 : next channel field info that would be great.
Note some of the fields have / in them so the current / would be bad.
Video fields:
Format
Codec ID
Codec ID/Hint
Audio fields:
Format
Format Version
Format profile
codec id (not hint)
This will also be helpful when new hardware not sigma comes out.
The text was updated successfully, but these errors were encountered: