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

Problems with non-asciil characters in paths #3424

Open
vondele opened this issue Apr 11, 2021 · 5 comments
Open

Problems with non-asciil characters in paths #3424

vondele opened this issue Apr 11, 2021 · 5 comments

Comments

@vondele
Copy link
Member

vondele commented Apr 11, 2021

an EvalFile cannot be used as a setoption if the path contains certain non-ascii characters.

for example setoption name EvalFile value C:\Users\Owner\Desktop\русский\nn-62ef826d1a6d.nnue does not work

similar issue in Leela, we might look there for solutions.. LeelaChessZero/lc0#1474

@snicolet
Copy link
Member

This stackoverflow post seems to have some related info: https://stackoverflow.com/questions/30829364/open-utf8-encoded-filename-in-c-windows

Not sure if we should care much about it in Stockfish

@AndrewTCEC
Copy link

AndrewTCEC commented Apr 11, 2021

As an actual finder of this problem, I say that I will wait as long as it takes to get it fixed and if you are not caring so much it's okay, it is a minor problem in the face of making the engine stronger. So take your time. I will adjust for ascii-only paths meanwhile. And another softening fact is that it takes place in SF-console only, using Fish in Arena does not cause any troubles (for some reason).

@dsmsgms
Copy link
Contributor

dsmsgms commented Apr 12, 2021

std::string is just bytes, Arena must be sending the correct encoding to standard input and console is using UTF-8.

@gvreuls
Copy link
Contributor

gvreuls commented Apr 15, 2021

It seems that UTF-8 console input is buggy at least for the dotnet runtime, if this is related I'm not sure we should code around it.

@dsmsgms
Copy link
Contributor

dsmsgms commented Mar 15, 2023

I cannot reproduce on this my system (Linux). Someone else has to check if we are going to find a potential fix. See link for an attempt. https://github.com/dsmsgms/Stockfish/tree/non_utf8_path/

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

No branches or pull requests

7 participants
@vondele @snicolet @gvreuls @dsmsgms @Disservin @AndrewTCEC and others