Add bbtools_tadpole, SeqSero2, and RGI #664
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@WhoisDonlee rgi_database_builder does not seem to be in the TS. The others are not a problem to install, but the rgi tools need a review or moved to IUC. |
This reverts commit 9cb8194.
|
The rgi_database_builder is installed when the rgi_wrapper is installed via the admin panel. I'm not that familiar with data managers, but it looks like it is included in the rgi_wrapper itself and not as a separate tool. |
|
Yes, which is also not good. I think we need to work in the rgi a bit. Do you know the author? |
|
Ah okay, for now I will remove it and will make it a separate PR once it's in the IUC. |
bgruening
approved these changes
Dec 20, 2023
|
Supi! This will be installed automatically on Sunday. Let me know if you need it before Xmas :) |
|
Awesome! Sunday will be fine! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The RGI wrapper and data manager is provided by card, which is also de developer of RGI. I wasn't sure if this was allowed so let me know if it belongs somewhere else.
I didn't run into any problems running it on our own instance.
https://github.com/arpcard/rgi_wrapper/tree/master