-
Notifications
You must be signed in to change notification settings - Fork 128
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
replace view with dump #450
base: master
Are you sure you want to change the base?
replace view with dump #450
Conversation
✅ Deploy Preview for nextflow-training ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not 100% sold on the dump thing, I quite often chuck views in as a lightweight thing in development. I have no intention of leaving them there, so dump isn't all that useful, and I have to do something extra on the command line.
... but I saw on Slack that this is the consensus view, so I'm fine with it.
``` | ||
|
||
As you can see, the flatten() operator has transformed the channel from containing arrays to containing individual elements. This can be useful when you want to process each item separately in your workflow. | ||
|
||
Remove the `.view()` operations before you continue. | ||
Remove the `.dump()` operations before you continue. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we don't need to with dump?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rephrased to make it optional. I am terrified of updating all line numbers everywhere.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm with @pinin4fjords, not 100% keen on dump replacing view here, it seems more work than necessary?
No description provided.