Skip to content

Conversation

@ErikDanielsson
Copy link
Contributor

@ErikDanielsson ErikDanielsson commented Aug 8, 2025

Added a blog post describing the changes made to the nf-core pipelines download command 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

@netlify
Copy link

netlify bot commented Aug 8, 2025

Deploy Preview for nf-core-main-site ready!

Name Link
🔨 Latest commit 19662f2
🔍 Latest deploy log https://app.netlify.com/projects/nf-core-main-site/deploys/68b02e7e043ce700080e7c5e
😎 Deploy Preview https://deploy-preview-3459--nf-core-main-site.netlify.app/blog/2025/refurbushing-the-pipeline-download
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions bot deleted a comment from netlify bot Aug 8, 2025
@ErikDanielsson ErikDanielsson marked this pull request as ready for review August 11, 2025 12:06
Copy link
Member

@mirpedrol mirpedrol left a 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!

@MatthiasZepper
Copy link
Member

Wouldn't that be a nicer image?

A truck carrying pipeline components

@ErikDanielsson
Copy link
Contributor Author

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

@MatthiasZepper
Copy link
Member

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!

ewels
ewels previously requested changes Aug 12, 2025
@ErikDanielsson
Copy link
Contributor Author

ErikDanielsson commented Aug 13, 2025

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 10s

Showing 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

@JulianFlesch JulianFlesch dismissed ewels’s stale review August 28, 2025 09:54

The review comment was addressed by the original PR author by rephrasing how Nextflow inspect finds containers. This change request is blocking

@mashehu mashehu merged commit 2cce8ff into nf-core:main Aug 28, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants