-
Notifications
You must be signed in to change notification settings - Fork 89
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
Add GPU CI via gitlab #796
Conversation
Maybe the last FAQ issue? In short, remove |
┌ Warning: The JULIA_MPI_BINARY environment variable is no longer used to configure the MPI binary.
│ Please use the MPIPreferences.jl package instead:
│
│ MPIPreferences.use_system_binary() # use the system binary
│ MPIPreferences.use_jll_binary() # use JLL binary
│
│ See https://juliaparallel.org/MPI.jl/stable/configuration/ for more details
│
│ ENV["JULIA_MPI_BINARY"] = "system"
│ MPIPreferences.binary = "MPICH_jll"
└ @ MPI ~/.julia/packages/MPI/5cAQG/src/MPI.jl:90 This is strange and shouldn't appear. In a fresh shell on Noctua 2 I get no warning:
I suggest you try to run your tests on the cluster outside of CI to see if you get the same output. I.e. use |
@carstenbauer Thanks for your help. To comment on your remarks:
Seems to have worked.
Yes I get the same output. Both with the tests I run in the gitlab runner as well as a simple |
Great.
Curious. I guess it doesn't really matter, it only opts to use an JLL MPI instead of the system one, but I might debug this when I find the time. If you want to fix it yourself (locally), simply following the instructions, i.e. creating a local preferences TOML via MPIPreferences.jl, should work. But I thought I had worked this out globally... |
BTW, coverage seems to have worked as well AFAICT. |
Ok, I think for now we won't do MPI and GPU, so I'll just ignore it. |
Sorry. I don't understand what you are referring to. It should basically work or it has worked on this setup successfully already? |
It, that is, the coverage.jl script, has worked already. If you look at the job output, at the very bottom, it says UPDATE: I see you have commented out the submit part. But I'm optimistic this works as well. |
FYI, Ich meine das hier ist die korrekte Badge für die CI: https://git.uni-paderborn.de/herbstm/DFTK.jl/badges/master/pipeline.svg?key_text=CI@PC2 |
Setup with the help of @carstenbauer.
@GVigne I will merge the dummy test first and you can add more sophisticated testing later.
TODOs: