Fix / report partition same as action slurm partition

A bug was reported by Ivana from the detector group.

Fix for running xfel-calibrate and building reports through exfel partition outside of xcal

Description

How Has This Been Tested?

Running xfel-calibrate with upex-middle and without. A report was generated fine and no errors.

Relevant Documents (optional)

sbatch: error: Batch job submission failed: User's group not permitted to use this partition

Types of changes

• Bug fix (non-breaking change which fixes an issue)

Checklist:

Reviewers

@schmidtp @danilevc @kluyvert

assigned to @ahmedk

added Waiting for review label

Ugh, that is an unfortunate side effect of that prior change.

I'm not really happy about adding another argument for that. If we really need to, we should also stick to the term finalize rather than report.

Can we use the same partition as for the processing job before it? For corrections and manual xfel-calibrate runs those are the same anyway, and it's not a problem if we queue the reports for dark processing as upex-high.

added 1 commit

• cc4eec7e - avoid using dedicated partition for reports

Compare with previous version

changed title from Fix / report{-slurm-}partition to switch requirements for xcal and users to Fix / report partition same as action slurm partition

changed the description

resolved all threads

This seems reasonable to me.

resolved all threads

added 1 commit

• ebd57c4f - repalce if/else with or

Compare with previous version

LGTM

removed Waiting for review label

Thank you for the review!

mentioned in commit f1b4a261

merged

resolved all threads