-
Notifications
You must be signed in to change notification settings - Fork 2
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
MAINT: Remove all functionality for AMRFinderPlus #106
MAINT: Remove all functionality for AMRFinderPlus #106
Conversation
This reverts commit b661340.
…okulich-lab#88)" This reverts commit 41bae5e.
…e called `fetch_amrfinderplus_db` (bokulich-lab#82)" This reverts commit 38e48b5.
…tionsDirFmt (bokulich-lab#92)" This reverts commit d785a0e.
This reverts commit 0b5439a.
This reverts commit fb9e4b5.
Hi @misialq |
Unfortunately, I have seen this in moshpit as well and as of today I have no clue what the problem is 😅 |
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.
This generally looks good - just a small clarifying question below.
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.
🚀
closes #105
Reverts all these commits:
"ENH: add new type
AMRFinderPlusDatabase
#81""ENH: add
AMRFinderPlusAnnotation
type #86""BUG: added database_format_version.txt to AMRFinderPlusAnnotationsDirFmt #92"
"ENH: Added new action that can download AMRFinderPlus database called
fetch_amrfinderplus_db
#82""ENH: Action to annotate MAGs and contigs with AMRFinderPlus #88"
"ENH: Removed
AMRFinderPlusAnnotation
type #95""ENH: Add three new parameters to run_amrfinderplus_n #97"
All this functionality was moved to q2-amrfinderplus.