-
Notifications
You must be signed in to change notification settings - Fork 3
Episode 03: Variant Calling Workflow Changes #3
Comments
@fpsom @crazyhottommy I reimplemented this variant calling workflow for one of my own samples yesterday and noticed that the samtools and bcftools versions are out of date. Right now the lesson uses version 0.1.19 for both tools. samtools is now on v 1.3.1 and bcftools is on 1.8. Can we move to the newer versions of the tools? I have code to do so. Note that bcftools v 0.1.19 does not have |
I definitely agree with that. And I would propose that all tools involved are upgraded to a later (if not the latest) version - i.e. The downside is that we'll probably need to upgrade the entire AWS image... |
Referencing this original issue. @fpsom @ErinBecker I have added exercises to 02-variant-calling so there are now three. Would you still like there to be exercises about the SAM format? |
Thanks @taylorreiter ! It looks good as is, but I would propose the following; we keep the two existing exercises as part of the episode, and we also include an exercise on the SAM format as an "advanced exercise". Knowing SAM format is in itself a challenge, so only people who are genuinely interested should be encourage to do it. And I wouldn't remove it altogether, as the format is the base input of a significant number of tools. |
Ok! I think that is a great idea. It will be a while before I have a chance to work on this, but I'll keep the issue open so we can address it. Thank you! |
Edits for Episode 03: Variant Calling Workflow
curl
.samtools mpileup
withbcftools mpileup
samtools mpileup
andbcftools
. See Issue #56 of original repobwa mem
instead ofbwa aln
. See also Issue #111 of original repoThe text was updated successfully, but these errors were encountered: