Add GPU labels to template
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 322
- Forks
- 255
- Avg merge
- 2d 3h
- Merged PRs (30d)
- 5
Description
Description of feature
Had discussion on slack with @ewels and Claus Hultschig whose Github Account I do not know.
Might not hurt to add some GPU labels that people can use. Note that for GPU support you need to either use a CUDA enabled base Docker container or add stuff yourself which still is kind of ugly.
Draft:
withLabel: with_all_gpus {
maxForks = 1
containerOptions = {
workflow.containerEngine == "singularity" ? '--nv':
( workflow.containerEngine == "docker" ? '-u $(id -u):$(id -g) --gpus all': null ) }
}
withLabel: with_single_gpu {
maxForks = 1
containerOptions = {
workflow.containerEngine == "singularity" ? '--nv':
( workflow.containerEngine == "docker" ? '-u $(id -u):$(id -g) --gpus device=0': null ) }
}
}
single-cpu needs the device to be parameterized.
Just opening this issue to keep track of this.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
No file or test is named. Start by locating the template or configuration entry point that handles withLabel blocks, then review how Docker and Singularity container options are generated. Done means usable all-GPU and single-GPU labels are supported, with the single-GPU device parameterized as requested.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- docker
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 30/100