-
Notifications
You must be signed in to change notification settings - Fork 10
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
Errors running/retraining Splendor using commands from tutorial #3
Comments
Thank you for testing my code, you're the first one from who I have feedback. Pretrained networks should now work, so current code should work. Just tested on head of master and it runs on my side. Some questions:
|
You're welcome, thank you for your quick reply.
Yes I have. I assume the following changes wrt master should be sufficient? pit.py:
main.py:
I didn't change it, so it's at 2.
Done, apparently I was at 1.17
I did as you suggested and playing now works, both with the pretrained models matching up as with a human player.
However, training still gives the same error:
|
I tried now with -V 74 and that seemed to work!
|
OK so:
FYI instead of starting training from scratch, you can load an existing checkpoint with the option |
onnxruntime v1.17 now supported on master |
I tried to play Splendor using the command from the tutorial (I first changed the package imports):
python ./pit.py splendor/pretrained_2players.pt human -n 1
But I got this following error:
So I figured maybe it's due to the mentioned issue "Ongoing code/features rework, some pretrained networks won't work anymore". So I reverted to the version of 30/1/2024, without avail. Then I decided to first run the training myself, using the example from the tutorial (I had to add the
-V 85
though, otherwise it complained about version 1 not existing):python main.py -m 800 -e 1000 -i 5 -F -c 2.5 -f 0.1 -T 10 -b 32 -l 0.0003 -p 1 -D 0.3 -C ../results/mytest -V 85
But now I got the following error:
The text was updated successfully, but these errors were encountered: