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
Make addSpecies optional also when using cut_its #372
Conversation
|
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 to me!
I have only comments/suggestions/questions that are optional, so I approve it right away.
path(fastafile) | ||
val(outfile) |
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.
It would be sort of more obvious and functional when
ampliseq/bin/add_full_sequence_to_taxfile.py
Lines 35 to 39 in f1abc20
outfile = taxfile.replace("ASV_ITS_", "ASV_") | |
# Join taxonomy and full sequence, write to file | |
tax = tax.set_index('ASV_ID').join(seqs.set_index('id'), how='outer') | |
tax.to_csv(outfile, sep='\t',na_rep="", index_label="ASV_ID") |
would actually use the value coming in here I think. But it works the way you are doing it right now I guess.
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.
It works, but you are right. Thanks for the suggestion!
PR checklist
nf-core lint
).nextflow run . -profile test,docker
).CHANGELOG.md
is updated.Comment
This adresses issue #370
Option --skip_dada_addspecies was not compatible with --cut_its, but now this should be fixed.