-
Notifications
You must be signed in to change notification settings - Fork 113
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
Re-add support for addspecies #245
Conversation
…emoved old links, removed one db, added a few others
…cludes adding it back in the test
…with the db's for qiime
|
conf/test.config
Outdated
@@ -23,6 +23,7 @@ params { | |||
input = "https://github.com/nf-core/test-datasets/raw/ampliseq/samplesheets/Samplesheet.tsv" | |||
metadata = "https://github.com/nf-core/test-datasets/raw/ampliseq/samplesheets/Metadata.tsv" | |||
dada_ref_taxonomy = "rdp=18" | |||
dada_ref_species = "rdp=18" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since this is only a band-aid on a not required parameter I wait until you have removed it completely.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, working on it!
…config and ampliseq.nf since it's not used
…ll configs, nextflow_schema and changelog
…rs, pacbio_its and multi to prevent taxonomy annotation in those cases
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
When I added the new way of accessing taxonomy-options, I did not think about the addspecies and realised I more or less just excluded it, especially from the test.config (I've re-added it there). But I'm not sure the way I handle it in ampliseq.nf is good - I did the same for
ch_dada_ref_species
as forch_dada_ref_taxonomy
butch_dada_ref_species
is never used. Maybe just change the--dada_ref_species
to just evaluate if it's true or not?