update slurm adapter for ORNL andes cluster

radical-cybertools / radical.saga

A Light-Weight Access Layer for Distributed Computing Infrastructure and Reference Implementation of the SAGA Python Language Bindings.

http://radical-cybertools.github.io/saga-python/

Other

83 stars 34 forks source link

update slurm adapter for ORNL andes cluster #823

Closed wjlei1990 closed 3 years ago

mtitov commented 3 years ago

hi @wjlei1990, can you please try to add 'andes' in self.rm.host.lower() to the line 577 instead of 583 (right after longhorn)?

wjlei1990 commented 3 years ago

For now, I don't set "--ntasks" or "-n" in my slurm header. I don't know if it is needed or not...

mtitov commented 3 years ago

RP will set a number of requested cores for this option (as an upper threshold). Don't have a strong opinion about this, since the following option --ntasks-per-node will be set (we don't have a resource config for Andes yet, but cores_per_node will be set there and that corresponding value will be used for --ntasks-per-node).