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

Contemplate an rslurm backend #120

Closed
wlandau-lilly opened this issue Oct 30, 2017 · 5 comments
Closed

Contemplate an rslurm backend #120

wlandau-lilly opened this issue Oct 30, 2017 · 5 comments

Comments

@wlandau-lilly
Copy link
Collaborator

Implementation would be similar to the future_lapply funcionality. @kendonB, do the examples in the rslurm vignette work for you? This would be the biggest reason for adding or not adding a separate backend. I do not want the drake code base to get too large, but I may add a special case if it makes or breaks a project that needs serious HPC.

@wlandau-lilly
Copy link
Collaborator Author

@kendonB I am particularly curious about your experience trying out slurm_apply().

@kendonB
Copy link
Contributor

kendonB commented Oct 30, 2017

It almost works: earthlab/rslurm#15

@kendonB
Copy link
Contributor

kendonB commented Oct 31, 2017

Got rslurm to work with the change suggested in earthlab/rslurm#15!

@wlandau-lilly
Copy link
Collaborator Author

Moot now that #115 is solved. Drake already has two completely independent ways to use SLURM, and that is plenty. But if someone has a burning desire to build in rslurm natively, I might be persuaded, and I will consider pull requests.

@wlandau-lilly
Copy link
Collaborator Author

wlandau-lilly commented Oct 31, 2017

@kendonB Sorry, I did not mean to sound dismissive of your efforts on rslurm. I am really glad you put in the time, and I am glad we have a backup. Nothing was wasted. Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants