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

GeneMarkS-2 #25

Closed
wants to merge 2 commits into from
Closed

GeneMarkS-2 #25

wants to merge 2 commits into from

Conversation

AaronRuben
Copy link
Contributor

Hi,

I just followed the style you used to for the incorporation of Prodigal.
Probably you will need to change the path for the gms2.pl executable. I didn't give any since I am not aware of licensing etc or how do you want solve this respectively.

Thank you very much!

Best,
Aaron

nigyta added a commit that referenced this pull request Feb 9, 2020
As per the pull request #25 and the issue #20, command-line options to enable GeneMarkS2, Diamond, RNAmmer have been added.
See [FAQ](docs/FAQ.md) for more details.

```
  --aligner STR         Aligner to use [ghostx(=default)|blastp|diamond]
  --use_genemarks2 STR  Use GeneMarkS2 to predict CDS instead of MGA. [auto|bact|arch]
  --use_rnammer STR     Use RNAmmer to predict rRNA instead of Barrnap. [bact|arch]
```
Note that GeneMarkS2, Diamond, RNAmmer are not bundled in the software package. If you want to use them, please install them by yourself.

Thank @AaronRuben for the PR for GeneMarkS2!
@nigyta
Copy link
Owner

nigyta commented Feb 10, 2020

PR is closed as the source codes have been merged according as per #26

@nigyta nigyta closed this Feb 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants