Splitting and grouping side quest #595
Conversation
First draft of a splitting and grouping side quest. This includes separating samples using filter, then grouping and spreading by intervals. Early version but introduces key operator concepts to participants.
This commit reverses the order to spread over intervals prior to grouping. This achieves two things: 1. It explains everything once and only once to make the tutorial simpler 2. It provides a real world reason for using groupTuple This makes the flow of the tutorial easier to understand, at the cost of very verbose outputs.
There was a problem hiding this comment.
Copilot reviewed 1 out of 4 changed files in this pull request and generated no comments.
Files not reviewed (3)
- side-quests/splitting_and_grouping/data/intervals.txt: Language not supported
- side-quests/splitting_and_grouping/data/samplesheet.csv: Language not supported
- side-quests/splitting_and_grouping/main.nf: Language not supported
✅ 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.
I quite like it.
For the introduction, I would vote for having a small picture illustrating the concept using the mail sorting. Maybe that's just me and the way I tend to memorise things.
Maybe it's because I just looked at the working with file PR, but it feels like, we could finish that module with splitCSV and then reuse it here. That way we could keep files and what to do with them a bit more separate?
Co-authored-by: Friederike Hanssen <[email protected]>
|
Looks like just one convo to resolve to be able to merge. I recommend creating an issue for the proposed followup. |
|
Oh and I forgot to say-- I had typed up some overall comments and lost them but this is really great! Very nice work @adamrtalbot ! |
First draft of a splitting and grouping side quest. This will likely be part 2 of a 4 part series:
This includes separating samples using filter, then spreading by intervals then grouping together by sample ID and interval (i.e. grouping replicates).
Bad bits so far:
normal_samples*?.fastq1)Good bits:
map