generated from CDCgov/template
-
Notifications
You must be signed in to change notification settings - Fork 33
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
QC step to exclude failed samples (based on CDC recommended FASTQ QC thresholds) from analysis. #125
Comments
Hi @jessres, thanks for your request. I'm seeing three potential solutions to this issue:
|
Hello @zmudge3 we appreciate such a quick response! Ideally option 3 is exactly what we are looking for. When do you expect v1.6 to be released? |
Got it, thank you. We're hoping to have it released by the end of the year. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is your feature request related to a problem? Please describe.
We ran into an issue where too many low coverage samples resulted in a pretty much empty vcf-to-fasta file, affecting the results of the passing samples and being unable to generate a phylogenetic tree. Further investigation shows that even 2 - 3 very low coverage samples can affect the accuracy of the phylogenetic tree.
Describe the solution you'd like
We would like to see failed low coverage samples be removed before vcf-to-fasta generation so that only passing samples are used for the core genome and results of the phylogenetic tree are accurate. QC results should still include all samples.
Describe alternatives you've considered
Alternatively, we have considered re-analyzing the run with just passing samples however, this negatively impacts our automated workflow and TAT.
Additional context
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered: