Skip to content

Add --slurm-partition CLI parameter to xfel-calibrate

Thomas Kluyver requested to merge config-slurm-partition into master

Description

We want to control the Slurm partition used by proposal and by action (correct vs dark). Making this a command line option for xfel-calibrate should allow it to be set in proposal YAML config files like slurm-partition: upex-high.

How Has This Been Tested?

I have tested launching xfel-calibrate with --slurm-partition upex-high. As expected, it passes --partition upex-high to sbatch, and then fails because I don't have permission to use that partition. I also checked that without suppplying --slurm-partition, it still submits to the exfel partition as before.

Types of changes

  • New feature (non-breaking change which adds functionality)

Checklist:

  • My code follows the code style of this project.

Reviewers

@schmidtp @ahmedk

Edited by Thomas Kluyver

Merge request reports