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

HPC testing failing on jean zay #229

Open
AnderGray opened this issue Oct 25, 2024 · 1 comment
Open

HPC testing failing on jean zay #229

AnderGray opened this issue Oct 25, 2024 · 1 comment

Comments

@AnderGray
Copy link
Collaborator

AnderGray commented Oct 25, 2024

Some clusters don't allow for --mem-per-cpu definitions. Easy fix by removing the following line from test/hpc/slurm.jl

"mem-per-cpu" => "100",

@AnderGray AnderGray added the bug Something isn't working label Oct 25, 2024
@FriesischScott
Copy link
Owner

I was thinking, maybe instead of verifying that the cluster connection works properly through the tests we export a function like this

function test_slurm_connection(account::String, parition::String),

which will run the same simple "external" model we do in the test now and inform the user that everything works as expected. This should also be easier for the user since no environment variables need to be set and it can be run independently from the rest of the tests. Thoughts?

@FriesischScott FriesischScott removed the bug Something isn't working label Nov 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants