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

signalP integration #61

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

signalP integration #61

wants to merge 9 commits into from

Conversation

tkosciol
Copy link
Member

signalP integration and tests

@RNAer
Copy link
Collaborator

RNAer commented Mar 15, 2016

have u heard anything back from SignalP people?

@tkosciol
Copy link
Member Author

Unfortunately not.
I will write a follow-up.

t.

On 15 Mar, 2016, at 4:05 PM, Zech Xu <[email protected]mailto:[email protected]> wrote:

have u heard anything back from SignalP people?


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHubhttps://github.com//pull/61#issuecomment-197062156

@tkosciol
Copy link
Member Author

I got a response saying that my request was forwarded to someone else. I hope they get back to me within a couple of days.

@tkosciol
Copy link
Member Author

I reached out SignalP people for the 4th time on Thursday (Apr 7).
I'd suggest that we consider our options within their license if we don't hear from them by Thursday Apr 14.

@RNAer
Copy link
Collaborator

RNAer commented Apr 21, 2016

the generated png files are not unit tested, right? Let's only include the files that's required by the tests.

@tkosciol
Copy link
Member Author

tkosciol commented Mar 3, 2018

guess this PR is obsolete now. should we just close it?

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