-
Notifications
You must be signed in to change notification settings - Fork 284
Automatic slurm allocations #794
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
base: master
Are you sure you want to change the base?
Conversation
321eb93
to
aad70d5
Compare
Pull Request Test Coverage Report for Build 13483409665Details
💛 - Coveralls |
aad70d5
to
d7dce98
Compare
Ok it turns out we need the following PRs to be merged first before this will actually work:
But looks like things are moving fast there which is good. Hopefully not too long. |
I should hopefully be able to get the remaining upstream work done soon-ish. |
Awesome. Thanks! |
0a133c9
to
88554ce
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #794 +/- ##
==========================================
- Coverage 93.54% 92.84% -0.70%
==========================================
Files 21 21
Lines 1565 1580 +15
==========================================
+ Hits 1464 1467 +3
- Misses 101 113 +12 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
This adds the
cluster_manager="slurm_native"
option which uses https://github.com/kleinhenz/SlurmClusterManager.jl. The idea with this is that you could potentially just haveand it will "just work" and set up the search over the cluster. This is the much more standard way to use slurm. This will get the right slurm configurations from the environment variables.
TODO:
Set up heap size hintWe don't actually need to do this! Since slurm will partition the memory for us and each Julia process will be aware.