-
Notifications
You must be signed in to change notification settings - Fork 0
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
Support analysis of Influenza B sequence data #23
Comments
This could be handled in a few ways:
We could also possibly combine aspects of multiple of these approaches. For example, we could take the dynamic approach by default, but allow the user to bypass that and "force" either Flu A or Flu B mode up-front. I think that may be the best approach. |
Possibility to provide a "sample sheet" that specifies which samples & controls should be handled as FluA and FluB? |
Thanks for that suggestion @stefkary. I think the place we'd like to handle that would be our nextflow wrapper for FluViewer, which is here: https://github.com/BCCDC-PHL/fluviewer-nf FluViewer itself is focused on analysis of a single sample at a time. In the nextflow wrapper would handle running analysis on multiple samples. I've added an issue there: BCCDC-PHL/fluviewer-nf#14 |
Ok @stefkary we've added support for samplesheet input on our BCCDC-PHL/fluviewer-nf pipeline. We're currently only collecting info on the sample ID and the R1 and R2 illumina fastq files through the samplesheet. Once we've incorporated the ability to specify a "Flu A mode" and "Flu B mode" via command-line arguments here we'll consider how we can incorporate that into the samplesheet. We'll also plan to support long (nanopore) read input via the samplesheet once that has been added here. |
The segment lengths as defined here:
FluViewer/fluviewer/fluviewer.py
Lines 302 to 311 in a71e39a
...and here:
FluViewer/fluviewer/fluviewer.py
Lines 1589 to 1591 in a71e39a
...are appropriate for Flu A sequences, but not for Flu B.
Adjust the segment lengths to be compatible with both FluA and FluB sequences.
The text was updated successfully, but these errors were encountered: