modules icon indicating copy to clipboard operation
modules copied to clipboard

Arity POC

Open nvnieuwk opened this issue 1 year ago • 2 comments

IMO the only thing stopping this for now is the inability for Nextflow to handle arity in optional inputs/outputs. I made an issue about this: https://github.com/nextflow-io/nextflow/issues/5111

nvnieuwk avatar Jul 03 '24 14:07 nvnieuwk

Do you want arity on every input?

Or just in use cases like these or even better where we expect 2 files?

edmundmiller avatar Jul 04 '24 13:07 edmundmiller

Do you want arity on every input?

Or just in use cases like these or even better where we expect 2 files?

I would prefer it on each input (this also has some self documenting purposes), but I'd wait until Nextflow supports arity for optional paths

nvnieuwk avatar Jul 04 '24 13:07 nvnieuwk

On the one hand, I like self documentation, but I think we shouldn't let this hold us back from at least adding arity: in the cases where paths are mandatory.

mahesh-panchal avatar Jul 10 '24 07:07 mahesh-panchal

Is this one of those module overhaul tasks?

mahesh-panchal avatar Jul 10 '24 07:07 mahesh-panchal

Is this one of those module overhaul tasks?

Not yet but I think it should be :)

nvnieuwk avatar Jul 10 '24 07:07 nvnieuwk

@nvnieuwk Do we leave this open until the issue is fixed on the Nextflow side?

famosab avatar Mar 11 '25 13:03 famosab

Yes please leave this open a bit longer, I'll try and chase this when I find some more time in the coming weeks/months

nvnieuwk avatar Mar 11 '25 14:03 nvnieuwk