-
Notifications
You must be signed in to change notification settings - Fork 954
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
Julia test failing (on Linux) #201
Comments
🤔 That's strange. I can't reproduce the error on my local machine or on Travis. Is there any other environment change on the build machine? |
That is certainly possible, but I'm not sure what it could be.. it has not changed much and has been up for much longer than a few days. My guess is still that either CxxWrap or something changed in the Julia packaging and the new one doesn't work with our system. Note that Travis does not build with Julia and hence does not test it (since it's an optional dependency). Can I confirm that this command would remove everything related to the custom Julia installation? (So I can retry from scratch)
|
Yes, that would be enough to remove all Julia related.
I can confirm that the latest release of CxxWrap is still
Yes, but from the log it did build with Julia with |
Hmm, you're right. I take it back: it's working on the Travis machines just not internally (or might just be my machine). This is not ideal, we don't really have a way to test it before pushing to github. I'll try again before the next update, but if you have any other ideas I'd love to hear them. |
@lanctot , I tried to search around and found a similar error with other packages here: So is your OS also Ubuntu 19.10? Could you try the workaround in the above link and see if it works for you?
I'm still using Ubuntu 18 so I can't reproduce the problem locally. I'll try in the docker environment and update what I can find here later. |
Ahh yes, I had to do that... right. And I think I got rid of that directory when we were testing other stuff, so I accidentally deleted it. Good catch! Ok, I'll try it on Monday when I go to do the update. |
Confirmed, that was indeed the problem. Thanks @findmyway! |
As Travis-CI is migrating, and will remove some options for OSS projects to get free workflows, we add github actions (we can remove travis CI afterwards). It will look like this: https://github.com/jblespiau/open_spiel-1/actions/runs/554623616 We only have a more recent version of MacOS, with Python 3.9. The current Travis do not specify the Python 3 version, while I explicitly pin it to be Python 3.7 here. I kept the versions the same as in .travis.yml otherwise (we have a Python 3.6, for Nox testing, we should likely upgrade it to be 3.7, but I would prefer this to be an orthogonal change. I just have a failure for the test with Julia: https://github.com/jblespiau/open_spiel-1/runs/1595264912?check_suite_focus=true It should be easy to fix this using in the install script: ``` # This is needed on Ubuntu 19.10 and above, see: # #201 if [[ "$JULIA_HOME" != "$HOME/packages/julias/julia-1.3.1" ]]; then if [[ -f /usr/lib/x86_64-linux-gnu/libstdc++.so.6 ]]; then echo "Julia not in $HOME. Copy libstdc++.so.6 (requires sudo)" sudo cp -f /usr/lib/x86_64-linux-gnu/libstdc++.so.6 $JULIA_HOME/lib/julia ``` but I simply disabled the test in this PR, and we can follow-up with that. PiperOrigin-RevId: 356724390 Change-Id: I69141b172c73333daaefd60534376e38dc36fdea
Hi @findmyway , seems that as of a few days ago the julia test is failing (oddly, only the test fails... it still compiles fine). This was the original error I got
Then when I removed julia and all the packages, starting from scratch (re-installing Julia via
./install.sh
) I get a different error:Did something change with the packages recently? Can you reproduce this?
The text was updated successfully, but these errors were encountered: