Enabling GPU usage for ribodetector in <#CE8SSJV3N...
# request-review
j
Enabling GPU usage for ribodetector in #CE8SSJV3N. Zero pipeline logic changes, all in module update + config + docs https://github.com/nf-core/rnaseq/pull/1790
approved with comments 1
m
Looks big, but easy to review
Wondering about this new params in case we add GPU support to other tools
j
There was a whole 'thing' recently, and that was discounted: https://nfcore.slack.com/archives/CQY2U5QU9/p1771579583230519
m
Ok
Oh yeah I remember now
j
There's complexity because the use cases are slightly different, you don't necessarily want to turn on GPU enablement across the board. Right now it's just this and parabricks, the case for a single setting may grow if that changes
Thanks for review!
m
Yeah, that's fine for me
I think we need proper tools selection handling that's consistent over pipelines
Better solutions than skip and run bolleans or huge tools or skip tools lists
And I think we're making progress along this way
👍 1