wlandau / crew.cluster

crew launcher plugins for traditional high-performance computing clusters

https://wlandau.github.io/crew.cluster

Other

26 stars 9 forks source link

[WIP] added missing 'partition' argument to slurm launcher #25

Closed kkmann closed 1 year ago

kkmann commented 1 year ago

Prework

[x] I understand and agree to the Contributor Code of Conduct.
[x] I have already submitted a discussion topic or issue to discuss my idea with the maintainer.

Related GitHub issues and pull requests

Ref: #24

Summary

Make the slurm partition configurable for crew_controller_slurm()

kkmann commented 1 year ago

Creation of the extended class work fine, but the map() function complains about non-matching character lengths on an attribute check. Did not get to the bottom of it.

wlandau commented 1 year ago

Creation of the extended class work fine, but the map() function complains about non-matching character lengths on an attribute check. Did not get to the bottom of it.

You might need development crew.

wlandau commented 1 year ago

I am actually gearing up for a set of releases, so I will merge this now. Thanks!