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

MAINT: Remove all functionality for AMRFinderPlus #106

Merged

Conversation

VinzentRisch
Copy link
Collaborator

@VinzentRisch VinzentRisch commented Sep 27, 2024

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.

@VinzentRisch
Copy link
Collaborator Author

Hi @misialq
The Ci is not running and its failing in the setup miniconda part. I have basically the same code as in the moshpit ci. The only difference is teh python version of 3.8 or 3.9.
Have you seen this before?
Error: Unable to locate executable file: /usr/share/miniconda/condabin/mamba. Please verify either the file path exists or the file can be found within a directory specified by the PATH environment variable. Also check the file mode to verify the file is executable.

@misialq
Copy link
Contributor

misialq commented Oct 1, 2024

Unfortunately, I have seen this in moshpit as well and as of today I have no clue what the problem is 😅

Copy link
Contributor

@misialq misialq left a 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.

q2_amr/card/mags.py Show resolved Hide resolved
Copy link
Contributor

@misialq misialq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@VinzentRisch VinzentRisch merged commit 6d140d9 into bokulich-lab:main Oct 4, 2024
2 checks passed
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

Successfully merging this pull request may close these issues.

MAINT: Remove everything related to AMRFinderplus
2 participants