Skip to content

Set MLC_RUN_STATE_SLURM env var for SLURM execution - #330

Merged
arjunsuresh merged 1 commit into
mlcommons:mainfrom
amd-arsuresh:patch-2
Oct 6, 2026
Merged

arjunsuresh merged 1 commit into
mlcommons:mainfrom
amd-arsuresh:patch-2

Conversation

@amd-arsuresh

Copy link
Copy Markdown
Contributor

Add environment variable for SLURM execution in run input.

✅ PR Checklist

✅ Testing & CI

  • Have tested the changes in my local environment, else have properly conveyed in the PR description
  • The change includes a GitHub Action to test the script(if it is possible to be added).
  • No existing GitHub Actions are failing because of this change.

📚 Documentation

  • README or help docs are updated for new features or changes.
  • CLI help messages are meaningful and complete.

📁 File Hygiene & Output Handling

  • No unintended files (e.g., logs, cache, temp files, pycache, output folders) are committed.

🛡️ Safety & Security

  • No secrets or credentials are committed.
  • Paths, shell commands, and environment handling are safe and portable.

🙌 Contribution Hygiene

  • PR title and description are concise and clearly state the purpose of the change.
  • Related issues (if any) are properly referenced using Fixes # or Closes #.
  • All reviewer feedback has been addressed.

Add environment variable for SLURM execution in run input.
@amd-arsuresh
amd-arsuresh requested a review from a team as a code owner October 3, 2026 09:34
@github-actions

github-actions Bot commented Oct 3, 2026

Copy link
Copy Markdown

MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅

@arjunsuresh
arjunsuresh merged commit d76f79f into mlcommons:main Oct 6, 2026
75 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants