-
Notifications
You must be signed in to change notification settings - Fork 39
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
Dockerfile not using latest version? #169
Comments
Hi Mirae, thank you for pointing out that. Haidong @HaidYi is responsible for updating the docker. Since he has multiple job interviews recently he does not have enough time to maintain the tool. You don't need to PR and you can update the dockerfile by yourside to update database. In addition, we already wrote dbcan_install.py for you to install the database. Thank you very much. |
Best of luck @HaidYi on the job interviews! Yea the comments about updating the Dockerfile are definitely not super pressing, might just be nice to fix eventually. Thanks for the tip about Another small thing - but if you all do update the image on Docker Hub in the future it might be nice to tag the various versions in order to make it easier to reference/refer back to a particular version of the software used. Thank you all again for this great package! :) |
Hello Mirae, I am getting back from the tough job search. Thank you for your suggestions. We have transferred the workflow from docker hub to the Again, thank you for your suggestions and stay safe! |
Hey all!
Thank you for putting this tool together!
Just dropping a note as I noticed that your
Dockerfile
seems to be making use of the v11 version of the db, while from the tutorial/website it seems like there is a v12 available. If you want I would be happy to open a PR updating it so it installs the latest version of the databases, but wanted to open an issue first in case this choice was intentional! If so may I ask why?Best,
Mirae
The text was updated successfully, but these errors were encountered: