search

cesmix-mit / LAMMPS.jl

MIT License
32 stars 11 forks source link

Automated checks error on windows-latest (julia 1.8 and 1.9) #55

Open Joroks opened 2 weeks ago

Joroks commented 2 weeks ago

I was able to fix test execution for julia 1.11 by removing unused test-dependencies. However I wasn't able to reproduce the errors for windows-latest (julia 1.8 and 1.9) and I don't know what's causing them.

vchuravy commented 2 weeks ago

So Windows memory management is wild and they essentially run a weak form of Valgrind all the time. So there is likely a use after free somewhere. Either in the wrapper or in Lammps itself.

Joroks commented 2 weeks ago

Ok I was now able to reproduce the error. It seems to be caused by the environment variable "OMP_NUM_THREADS" not being set. If I either don't set that variable or set it to "1", issuing a run command will cause the error. If I set it to "2" or higher, it doesn't error.

I can also only get it to error when running the pgk"test". It doesn't happen when I include the runtests.jl file