-
Notifications
You must be signed in to change notification settings - Fork 1
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
MediaConch Policy Check "Failed" #51
Comments
This is weird. Might take some more digging. I have General/Title and General/Description set to just check to make sure they "exist". There shouldn't be a check for General/Encoded_Application at all. Is the md5 of your xml file: 4f3c5e0a0853cef691b40b440c81d2c2 ? |
circling back to this, I figured out why the "Title" field is failing. MediaConch is seeing the custom field in 2012_79_2_224_1a_PM.mkv as: General/extra[1]/title But in JPC_AV_0500.mkv and 2012_79_2_230_1a_PM.mkv have the title in General/Title instead, this is what the mediaconch policy is looking for. None of the other test files I have (JPC_AV_00011, JPC_AV_01663, JPC_AV_1709, and JPC_AV_1710) have either title field. I propose we remove the Title check from the mediaconch policy all together since we now check for it using mediatrace. Speaking of, the mediatrace check actually fails for 2012_79_2_224_1a_PM.mkv too, because it is looking for "TITLE" and not "title". I will make mediatrace checks case insensitive going forward. |
I've added a new mediaconch policy, JPC_AV_NTSC_MKV_2024-07-31.xml, which does not check for Title, Description or Encoded_Application/Encoded_By, and updated the command_config.yaml file to call the new file. |
Hello, @eddycolloton !
An oddity that may (or may not) be related is that I've been getting this message upon finishing every transfer:
Dave Rice says "the matroska didn't finish writing properly. Normally the muxer writes all the audiovisual data and then goes back and adds the crc's and cues and some other data when it's finalizing it, but that last step didn't happen." Based on his advice, I've been running |
Thanks @DSohl . Are these PCM audio encoded? The MediaInfo and Exiftool checks accept both codecs, so that's why we're getting fails for those in MediaConch and not in the other checks. Could be worth discussing removing those audio codec checks from the mediaconch policy if both PCM and FLAC are acceptable for JPC mkv files. @BleakleyMcD God I hate aspect ratio math. Could be a few different things going on here:
What do you think? Would you mind sending me this file? |
@eddycolloton Yeh, they're 24-bit PCM. |
Thanks @DSohl , I received both versions of |
The two versions of JPC_AV_01772 both have the same display aspect ratio and pixel aspect ratio, which is what is failing on our mediaconch policy. This would appear to be different than what Dave is talking about with the vrecord error message. I'm a bit stumped on what the correct pixel aspect ratio (PAR) value should be for an NTSC file. This is the resource I see cited on this subject most often: https://bavc.org/par-sar-and-dar-making-sense-standard-definition-sd-video-pixels/ This says that Rec 601 standardizes the PAR for NTSC as 10:11, which is 0.90 repeating. So, does that mean the PAR should be 0.900 or 0.909? I haven't been able to find public policies that state a specific PAR, most recommendations and standards just say things like "non-square" or "Rec 601 compliant". The display aspect ratio being 400:297 makes me think the PAR may be incorrect, and that the mediaconch policy is correctly flagging this as encoded incorrectly. But again, I find the whole aspect ratio math thing exceedingly confusing so I'm not the best judge. |
Hi @EmCNabs @BleakleyMcD @DSohl - I'm looking to close out this issue. To summarize the thread so far:
Pending questions:
|
Just updated to JPC_AV_NTSC_MKV_2024-09-20.xml which makes PCM the passing audio codec value 710d170 |
MKV files report back as "failed" using the MediaConch JPC_AV_NTSC_MKV_2023-11-21.xml policy because it is checking for a specified Title & Descripton, and an older version of Lavf. (Full MC report attached)
General/Title is Black Film Archives, Tape 3 ❌ fail (Actual: Midnight Ramble: Oscar Micheaux Clips)
Value: Title
Tracktype: General
Operator: =
Requested: Black Film Archives, Tape 3
Actual: Midnight Ramble: Oscar Micheaux Clips
Xpath: mi:MediaInfo/mi:track[@type='General'][*]/mi:Title
General/Description is A man demonstrates film winding on a film bench; a wide angle shot outside of Southern Methodist University ❌ fail
Value: Description
Tracktype: General
Operator: exists
Requested: A man demonstrates film winding on a film bench; a wide angle shot outside of Southern Methodist University
Xpath: mi:MediaInfo/mi:track[@type='General'][*]/mi:Description
General/Encoded_Application is Lavf59.27.100 ❌ fail (Actual: Lavf60.3.100)
Value: Encoded_Application
Tracktype: General
Operator: =
Requested: Lavf59.27.100
Actual: Lavf60.3.100
Xpath: mi:MediaInfo/mi:track[@type='General'][*]/mi:Encoded_Application
2012_79_2_224_1a_PM.mkv_MediaConchReport_FAIL.txt
The text was updated successfully, but these errors were encountered: