Skip to content

Fix gpus-per-task is mutually exclusive with tres-per-task error. #16

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Jun 18, 2024

Conversation

joaander
Copy link
Member

@joaander joaander commented Jun 17, 2024

Description

  • Fix --mem-per-gpu on Great Lakes.
  • Use --tres-per-task=gpu: instead of --gpus-per-task=

Motivation and context

Allow users to submit GPU jobs with row.

How has this been tested?

validate.py tests pass on:

  • Great Lakes
  • Delta
  • Anvil
  • Andes
  • Frontier

Checklist:

  • I have reviewed the Contributor Guidelines.
  • I agree with the terms of the Row Contributor Agreement.
  • My name is on the list of contributors (doc/src/contributors.md) in the pull request source branch.
  • I have added a change log entry to doc/src/release-notes.md.

joaander added 3 commits June 17, 2024 11:39
Recent versions of slurm set the `SLURM_TRES_PER_TASK` environment
variable which conflicts with `--gpus-per-task`. Set `--tres-per-task`
instead to avoid `gpus-per-task is mutually exclusive with tres-per-task`
errors.
@joaander joaander merged commit d373804 into trunk Jun 18, 2024
17 checks passed
@joaander joaander deleted the fix-gpu-errors branch June 18, 2024 14:36
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.

1 participant