llnl / llnl/maestrowf

flux: `gpus` key semantics should be total number of gpus

Open
#360 1 comment 0 reactions 1 assignee Claimed by @FrankD412 View on GitHub
Adapters enhancement Specification
Dominant language
Python
Stars
161
Forks
49
PR merge metrics
No merged PRs in 30d

Description

In the 0.26.0 adaptor, parts of it assume that `gpus` means total number of gpus and other parts assume it means `gpus_per_task`. The adaptor should be modified to assume it means total number of gpus for consistency with the slurm adaptor

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.