nextflow icon indicating copy to clipboard operation
nextflow copied to clipboard

Deprecate tap operator, extend set operator to return input channel

Open bentsherman opened this issue 2 years ago • 2 comments

The tap isn't marked as deprecated, but it depends on Channel.create() which is deprecated. I tried the top example with Channel.empty() instead but it didn't work, which makes sense because you shouldn't be able to just write to a channel like that.

So should we deprecate tap ? It seems like we could achieve the same functionality by changing set to return itself like view. Then you could use set in the middle of an operator chain to emit into a new channel while proceeding with other channel operators.

bentsherman avatar Jun 20 '22 13:06 bentsherman

another option could to allow tap to create an new channel by name, e.g.

something.tap('new_channel')

but not sure it makes much sense

pditommaso avatar Jun 20 '22 15:06 pditommaso

Related #1766

bentsherman avatar Jul 13 '22 23:07 bentsherman

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Dec 21 '22 01:12 stale[bot]

Bump

pditommaso avatar Dec 21 '22 07:12 pditommaso