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

[Request An Update]: kaptive 2.0.7 + V. parahaemolyticus databases #661

Closed
kapsakcj opened this issue May 3, 2023 · 4 comments · May be fixed by #694
Closed

[Request An Update]: kaptive 2.0.7 + V. parahaemolyticus databases #661

kapsakcj opened this issue May 3, 2023 · 4 comments · May be fixed by #694
Labels
enhancement New feature or request

Comments

@kapsakcj
Copy link
Collaborator

kapsakcj commented May 3, 2023

What container needs an update?

upgrade to kaptive 2.0.7 and add capability to serotype V. parahaemolyticus using newly-released databases

original tweet that I saw: https://twitter.com/DrKatHolt/status/1653795878077968386

publication: https://www.microbiologyresearch.org/content/journal/mgen/10.1099/mgen.0.001007

github repo for where V. parahaemolyticus databases live: https://github.com/aldertzomer/vibrio_parahaemolyticus_genomoserotyping

would be good to also add a test to the test stage that downloads a V. parahaemolyticus genome and runs kaptive on it with the new databases

@kapsakcj kapsakcj added the enhancement New feature or request label May 3, 2023
@kapsakcj
Copy link
Collaborator Author

kapsakcj commented May 3, 2023

also note that the github organization for Kaptive has changed - this will need to be taken into account when writing the dockerfile:

https://github.com/klebgenomics/Kaptive/releases

@kapsakcj
Copy link
Collaborator Author

kapsakcj commented May 3, 2023

Started a dev branch cjk-kaptive-upgrade with the dockerfile & README.md in progress

@kapsakcj
Copy link
Collaborator Author

kapsakcj commented May 4, 2023

dockerfile is in a stable place, don't think I have many more changes to make. the docker image builds successfully and tests against K.pneumo, A.baum, and Vparahaem

Last piece is to add information on the custom database to /kaptive/2.0.7/README.md

@kapsakcj
Copy link
Collaborator Author

Opened PR #694 to address this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Completed/Closed
Development

Successfully merging a pull request may close this issue.

2 participants