-
Notifications
You must be signed in to change notification settings - Fork 0
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
App returns error when attempting to fix the banner for a ROM with special characters on certain locales #7
Comments
That does sound like a proper error. I'll try to replicate it and let you know when it is fixed. |
Some more details on the issue. It looks like the problem is not the coloring, but rather the used GBA ROM. The problem I've mentioned happened only on the .cia files created using the Pokemon games for the NTSC region. I'll try to play around a little bit with other ROMs, and post here my findings |
I can't seem to replicate this problem. I tried a NTSC version of Emerald both with the regular and a colored cartridge and both get fixed just fine. How are you running the banner fixer? Are the CIAs in the same folder as the program? Are you running by double clicking or via command line? With what arguments if any? |
Hmm... Try changing the publisher name within NSUI also, as the way I am running that, I am pasting a bunch of .cia files at the root of the project, and run the program |
I recreated your screenshot as close as possible, still works for me. Can you try running via command line? Open the folder containing nsui_banner_fixer.exe/.py and the .cia files, in the explorer address bar type |
@pivotiiii Here is the output
I do not know if that will be of any help, but I could also send the .cia that I am using currently |
@pivotiiii On a second look, the issue seems to be the .cia filename, I am using there a special character in the Good to know that the problem is so minor 😄 |
The |
Most probable, the PC I was running the whole job on was acquired roughly a month ago, and I've changed the locale as it had issues displaying Cyrillic characters, maybe that somehow messed the logic of some other apps, as initially the OS was installed using EN-US. I will be closing the issue then, as it mostly looks like a problem with my PC, than an issue with the app itself |
This still shouldn't be happening, I'll reopen this just to remind myself to strip any special characters in general. |
Hey, I just uploaded a new version which was rewritten in C++. From my (very limited) testing is seems to properly handle special characters, but then again so did the Python version on my machine/locale. It would be nice if you could check how it works for you, thanks :) You can get it here. |
Sure, will take a look at it today. Will give a response upon the result tomorrow, in the worst case |
@pivotiiii It looks like there is another issue with this version
Ping me if you'll need any additional info |
It's the same problem. I guess I'll have to bite the bullet and set up a VM for checking different locales at some point. |
I have a couple of ROMs that I've modified the cartridge color within NSUI
When I try to supply such .cia files to the banner fixer, I am getting the following error
All the other .cia files that I've supplied to the banner fixer worked fine, only the ones with modified colors failed, so the problem is not in the version of the NSUI app used for generating the said .cia files
The text was updated successfully, but these errors were encountered: