Skip to content
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

rename amplicon16S_analysis to classify_qiime2_multi #463

Open
dpark01 opened this issue Apr 13, 2023 · 0 comments
Open

rename amplicon16S_analysis to classify_qiime2_multi #463

dpark01 opened this issue Apr 13, 2023 · 0 comments
Assignees

Comments

@dpark01
Copy link
Member

dpark01 commented Apr 13, 2023

I've been increasingly thinking that maybe we should rename our new 16S workflow to be more consistent with the naming schemes of other WDL workflows we have in here. classify_qiime2_multi would fit the verb_toolname_entitytype naming scheme we have for a lot of our workflows already.

The thing that makes it discordant from all our other classify_xxx workflows is that it doesn't emit 1) krona plots, 2) kraken-style summary reports, or 3) kraken-style per-read taxid assignments. But that's okay, especially since it looks like 1 is not hard to add with the right plugin and even if we didn't add that functionality, it would at least help bucket the workflow with the right family of related workflows.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants