-
Notifications
You must be signed in to change notification settings - Fork 253
Download refactor blog post #3459
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
Conversation
✅ Deploy Preview for nf-core-main-site ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
3a9bbb8 to
c8cc16c
Compare
466b752 to
6ae1401
Compare
mirpedrol
left a comment
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.
Added a couple of comments, but otherwise looks good!
sites/main-site/src/content/blog/2025/refurbushing-the-pipeline-download.mdx
Outdated
Show resolved
Hide resolved
sites/main-site/src/content/blog/2025/refurbushing-the-pipeline-download.mdx
Outdated
Show resolved
Hide resolved
sites/main-site/src/content/blog/2025/refurbushing-the-pipeline-download.mdx
Outdated
Show resolved
Hide resolved
sites/main-site/src/content/blog/2025/refurbushing-the-pipeline-download.mdx
Outdated
Show resolved
Hide resolved
Co-authored-by: Júlia Mir Pedrol <mirp.julia@gmail.com>
|
Wouldn't that be a nicer image? |
I completely forgot about updating the header image, thanks for pointing that out! I'll look into some different ideas, but I might stick with yours |
Feel free to pick what you like! I just felt the rocket was a bit boring, since it had been used before and also had not really ties to your PR. Therefore, I quickly searched and found this "pipeline on the move" image, but a container vessel or whatever will certainly also work! |
sites/main-site/src/content/blog/2025/refurbushing-the-pipeline-download.mdx
Outdated
Show resolved
Hide resolved
sites/main-site/src/assets/images/blog/pipeline-download-refactor/rnaseq-in-action.png
Outdated
Show resolved
Hide resolved
|
Gifs are created with vhs. If anyone is interested in reproducing or tweaking the outputed gifs, here are the corresponding tape files. Note that I've used the branch in nf-core/tools#3706 to generate the videos, so it shows that the user should use load scripts for Docker and Podman rather than just outputting the Docker load command Showing fail with old Nextflow version Output nf-old-demo.gif
Set Framerate 15
Set PlaybackSpeed 1.0 # Make output 2 times faster
Set Shell bash
Set FontSize 30
Set Width 1800
Set Height 900
Type "nf-core pipelines download bamtofastq \"
Sleep 1510ms
Enter
Tab
Type "--revision 2.2.0 \"
Sleep 1500ms
Enter
Tab
Type "--container-system singularity"
Sleep 1500ms
Enter
Sleep 10sShowing docker downloads Output docker-demo.gif
Set Framerate 15
Set PlaybackSpeed 2.0 # Make output 2 times faster
Set Shell bash
Set FontSize 30
Set Width 1800
Set Height 900
Type "nf-core pipelines download bamtofastq \"
Sleep 1510ms
Enter
Tab
Type "--revision 2.2.0 \"
Sleep 1500ms
Enter
Tab
Type "--container-system docker \"
Sleep 1505ms
Enter
Tab
Type@200ms "--compress none"
Sleep 1500ms
Enter
Sleep 120s |
…re into download-blog-post
sites/main-site/src/content/blog/2025/refurbushing-the-pipeline-download.mdx
Outdated
Show resolved
Hide resolved
sites/main-site/src/content/blog/2025/refurbushing-the-pipeline-download.mdx
Outdated
Show resolved
Hide resolved
sites/main-site/src/content/blog/2025/refurbushing-the-pipeline-download.mdx
Outdated
Show resolved
Hide resolved
sites/main-site/src/content/blog/2025/refurbushing-the-pipeline-download.mdx
Outdated
Show resolved
Hide resolved
sites/main-site/src/content/blog/2025/refurbushing-the-pipeline-download.mdx
Outdated
Show resolved
Hide resolved
sites/main-site/src/content/blog/2025/refurbushing-the-pipeline-download.mdx
Outdated
Show resolved
Hide resolved
The review comment was addressed by the original PR author by rephrasing how Nextflow inspect finds containers. This change request is blocking
Added a blog post describing the changes made to the
nf-core pipelines downloadcommand in nf-core/tools#3634 and related PRs. Once this is ready we should be able to merge the tool PRs.@netlify /blog/2025/refurbushing-the-pipeline-download