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

Include an optional ORF prediction step #15

Closed
iimog opened this issue Apr 21, 2016 · 4 comments
Closed

Include an optional ORF prediction step #15

iimog opened this issue Apr 21, 2016 · 4 comments
Milestone

Comments

@iimog
Copy link
Member

iimog commented Apr 21, 2016

Having an optional ORF prediction step (using established software like getorf from EMBOSS) would allow using assembled nucleotide sequences as input. Therefore the requirement to annotate genomes first could be dropped. It is also possible to auto-detect if fasta files contain aa or nt sequences. So even mixed input without explicit specification would be possible.

@iimog iimog added this to the 1.1.0 milestone Apr 21, 2016
@iimog iimog removed this from the 1.1.0 milestone Mar 6, 2017
@jdguzmanv
Copy link

Would be really great to have bcgTree 2.0 working also on nucleotide level. Input could be an annotated Genbank file, or a nucleotide genome fasta file and bcgTree calling an a fifth tool such as Prokka or Prodigal for annotation.

@iimog
Copy link
Member Author

iimog commented Oct 20, 2021

Thanks for the feedback. I agree that this would be a very useful feature. I'll have another look at it to see if it can be quickly implemented. I'll keep you posted.

iimog added a commit that referenced this issue Oct 22, 2021
@iimog
Copy link
Member Author

iimog commented Oct 22, 2021

Hey @jdguzmanv I implemented an optional translation step. You can now provide fasta files either via --proteome as peptide fasta files or via --genome as nucleotide fasta files. Each fasta file that you provide as --genome is first translated with prodigal (which needs to be installed and available). I also added the new options to the GUI.

It is not merged into the master branch yet, so please have a look in the feat-nuc2pep branch: https://github.com/molbiodiv/bcgTree/tree/feat-nuc2pep
Let me know what you think. I'll work on polishing and merging this feature next week.

@iimog iimog added this to the 1.2.0 milestone Oct 25, 2021
@iimog iimog removed the 0 - Backlog label Oct 25, 2021
@iimog
Copy link
Member Author

iimog commented Oct 27, 2021

This feature is part of the latest release (v1.2.0 released a couple of minutes ago). Please try it and let me know if you have questions or suggestions for improvement.

@iimog iimog closed this as completed Oct 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants