Skip to content

Conversation

christopher-hakkaart
Copy link
Collaborator

@christopher-hakkaart christopher-hakkaart commented Oct 2, 2025

  • Add a rnaseq-nf tutorial to act as a prerequisite for other guides and tutorials that reference it heavily.

Signed-off-by: Christopher Hakkaart <[email protected]>
Signed-off-by: Christopher Hakkaart <[email protected]>
Copy link

netlify bot commented Oct 2, 2025

Deploy Preview for nextflow-docs-staging ready!

Name Link
🔨 Latest commit 7fa5c82
🔍 Latest deploy log https://app.netlify.com/projects/nextflow-docs-staging/deploys/68e6db8a725ea3000817638a
😎 Deploy Preview https://deploy-preview-6440--nextflow-docs-staging.netlify.app
📱 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.

Signed-off-by: Christopher Hakkaart <[email protected]>
Signed-off-by: Christopher Hakkaart <[email protected]>
@christopher-hakkaart christopher-hakkaart changed the title Add rnaseq-nf tutorial docs: Add rnaseq-nf tutorial Oct 2, 2025
@christopher-hakkaart christopher-hakkaart marked this pull request as ready for review October 2, 2025 20:35
@christopher-hakkaart christopher-hakkaart requested a review from a team as a code owner October 2, 2025 20:35
@christopher-hakkaart
Copy link
Collaborator Author

@bentsherman - This is what I was thinking for a precursor for many of the guides/tutorials in development. I've tried to keep it reasonably high level, but enough to explain what runs, and where it can be found.

I was thinking about linking to the specific files in the repo, i.e., when referring to the entry workflow or module, link to those in GitHub.

Let me know what you think.

christopher-hakkaart and others added 7 commits October 3, 2025 09:40
Signed-off-by: Christopher Hakkaart <[email protected]>
Signed-off-by: Christopher Hakkaart <[email protected]>
Signed-off-by: Christopher Hakkaart <[email protected]>
Signed-off-by: Christopher Hakkaart <[email protected]>
Signed-off-by: Christopher Hakkaart <[email protected]>
Signed-off-by: Ben Sherman <[email protected]>
@bentsherman
Copy link
Member

Thanks Chris, looks good so far.

I just updated the rnaseq-nf source code to align better with the initial version of the workflow outputs tutorial, and I updated this PR accordingly.

However now I think I might have updated rnaseq-nf "too much" by joining the fastqc/quant results into a single channel. That ends up making the workflow outputs tutorial "too simple." So I might revert rnaseq-nf back to having separate fastqc/quant channels so that the tutorial can still show how to join them to prepare for workflow outputs. Need to think a bit more about this

@christopher-hakkaart
Copy link
Collaborator Author

Nice! The changes look great. I see what you mean about the separate fastqc/quant channels. I think it would be beneficial to still display this, as it's likely a common use case and could be considered a learning outcome.

I'm going to explore presenting code in the Migrating to workflow outputs as separate code blocks or tabs, labeled or tagged to indicate their origin. It might look terrible, be unclear, and/or disrupt the flow, so I may not proceed with it further. I'm just flagging that I may create a draft PR to show what it looks like.

Signed-off-by: Ben Sherman <[email protected]>
@bentsherman
Copy link
Member

rnaseq-nf and tutorials should be up to date now with separate fastqc/quant channels

@christopher-hakkaart
Copy link
Collaborator Author

It's all looking really good. Did you want to let it sit and think about it some more? Or are we close to merging?

@bentsherman
Copy link
Member

Yeah I'm happy to merge

@christopher-hakkaart christopher-hakkaart merged commit a830125 into nextflow-io:master Oct 9, 2025
25 checks passed
@christopher-hakkaart christopher-hakkaart deleted the docs-rnaseq-nf-tutorial branch October 9, 2025 03:20
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.

2 participants