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

"System.FormatException: The input string format is incorrect" #13

Closed
TheWonderfulTartiflette opened this issue May 21, 2023 · 10 comments
Closed

Comments

@TheWonderfulTartiflette

Using 2.1.3, I'm getting this error. I've linked the screenshot of it (parts of it are in French)
image
Both files are in .mp4, the source is in H264 and the compressed one is in AV1

@toutrob
Copy link

toutrob commented Jan 27, 2024

I have exactly the same error with 2 .mkv files in H264. My Windows is also in French, maybe that's something to do with that?

@ThatNerdUKnow
Copy link
Owner

The problem seems to be with the part where I'm reading the xml file generated by vmaf to show the results. Could you share a copy of an xml file generated by vmaf-gui? If I remember correctly they should be getting generated in the root directory of the executable

@toutrob
Copy link

toutrob commented Jan 27, 2024

@ThatNerdUKnow
Copy link
Owner

One more question, what's the absolute path to that xml file on your drive?

@toutrob
Copy link

toutrob commented Jan 27, 2024

C:\Users\robin\Downloads\vmaf-gui.2.1.3\Release

@ThatNerdUKnow
Copy link
Owner

Strange, I wouldn't think that path would cause a format exception. I'll need to investigate what can cause Xdocument.Load to throw a formatexception

@ThatNerdUKnow
Copy link
Owner

If I force vmaf-gui to skip all processing and load the XML @toutrob provided, it seems to load just fine. That could possibly rule out the formatting of the XML document. I do also wonder if this is a character encoding thing, perhaps the behavior of XDocument.load depends on the locale of the system?

@ThatNerdUKnow
Copy link
Owner

Okay, I think I've got it. Apparently the french locale uses commas instead of periods to represent decimal points. It looks like I should be able to override the locale that XDocument is using to load the xml file, so I'm going to give that a try.

@ThatNerdUKnow
Copy link
Owner

@toutrob , can you please try using this release?
https://github.com/ThatNerdUKnow/vmaf-gui/releases/tag/2.1.4

@toutrob
Copy link

toutrob commented Jan 28, 2024

It works perfectly for me, thank you for your work!

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

No branches or pull requests

3 participants