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

Exception thrown by method called. #27

Open
Shalupon opened this issue Jul 31, 2024 · 7 comments
Open

Exception thrown by method called. #27

Shalupon opened this issue Jul 31, 2024 · 7 comments

Comments

@Shalupon
Copy link

image
Yeah, this happened. Tried multiple versions of vegas, all the same.

@Cantersoft
Copy link
Owner

Please send the MIDI file that you used so I can test it on my system.

@Shalupon
Copy link
Author

Shalupon commented Aug 1, 2024

https://github.com/user-attachments/assets/fda752b6-0ea1-4e6d-b4c6-f424fc175fb2
I'm not sure the midi files are involved here. I forgot to tune them to A4 in the vid, but I tried it again afterwads

@Shalupon Shalupon closed this as not planned Won't fix, can't repro, duplicate, stale Aug 1, 2024
@Shalupon Shalupon reopened this Aug 1, 2024
@Cantersoft
Copy link
Owner

Cantersoft commented Aug 3, 2024

Considering that YTPMVE.cs is in the wrong location in the Script Menu, something must have gone wrong in the installation process. If you followed the manual steps written in the README, I will have to apologize to you, because they were outdated.

Currently, the correct process is to move YTPMVE.cs, Newtonsoft.Json.dll, and YTPMVE.cs.config into %ProgramFiles%\VEGAS\VEGAS Pro XX.0\Script Menu\YTPMVE_VEGAS, and hoof.ico, YTPMVE.py, YTPMVE_UI.exe, and YTPMVE_UI.pyw to %ProgramFiles%\VEGAS\YTPMVE\.

@Shalupon
Copy link
Author

Shalupon commented Aug 4, 2024

image
image
image

I have reinstalled it according to your instructions, but it appears that nothing have changed. I've provided the folders and the error, I think it's the same.

@Cantersoft
Copy link
Owner

After conducting a bit of research, I discovered that apparently, numbers register differently on some Windows systems that are not in the English language! I'll spin up a Windows virtual machine in Russian and perform some code testing on it now that I realize this is an issue. It will take me some time. I need to add another parameter, IFormatProvider, to some of the parsing code.

@Shalupon
Copy link
Author

Oh, that explains it, I'll wait then. Good luck.

@Cantersoft
Copy link
Owner

Okay, I got around to fixing this problem at last. Adding CultureInfo.InvariantCulture as an argument to every parse line prevents the parser from behaving erroneously. Tested on a Russian language Windows 10 machine with success. https://github.com/Cantersoft/YTPMVE/releases

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

2 participants