Skip to content

fix: pass slurm account to inference srun call - #262

Merged
lclanzi merged 1 commit into
mainfrom
fix/inference-srun-account
Sep 11, 2026
Merged

lclanzi merged 1 commit into
mainfrom
fix/inference-srun-account

Conversation

@lclanzi

@lclanzi lclanzi commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • The inference_execute rule runs anemoi-inference via a nested srun inside a squashfs-mount ... bash -c block, separate from snakemake's own job submission
  • slurm_account (added in Expose SLURM account and GPUs from config #180) was only wired into snakemake's --default-resources, so it never reached this inner srun call. This meant that GPU inference jobs ran under the cluster's default account instead of the one configured in profile.default_resources.slurm_account
  • Adds an account_flag param that renders --account=<slurm_account> when configured, or an empty string otherwise, and passes it to the inner srun call.

@lclanzi
lclanzi requested review from dnerini and frazane September 9, 2026 15:49

@frazane frazane left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@lclanzi
lclanzi merged commit 83a6009 into main Sep 11, 2026
4 checks passed
@lclanzi
lclanzi deleted the fix/inference-srun-account branch September 11, 2026 06:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants