-
Notifications
You must be signed in to change notification settings - Fork 0
feat: add prokka and bakta for annotation #4
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
Conversation
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.
- pgap will fail, if the full path is not defined in the output of
prepare_yaml_files. The full path is required otherwise pgap will not work. - add the option to be able to define the path to a bakta db that is already downloaded
- add functionality to run annotation with multiple tools
|
Hi Rina, point 2 and 3 I get, but for point 1 with PGAP paths I don't really know what you mean. I've not changed the PGAP logic as far as I'm aware. But I will test. |
rabioinf
left a comment
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.
looks fine now.
closes #3 , closes #5, closes #6, closes #7