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

Adds -F option to hmmc2 to report hits as Stockholm format MSA for an… #272

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

foreveremain
Copy link

PR is a result of discussions with @biomadeira

EBI Web services currently do not support download of the hits for an hmmsearch as an MSA. This PR adds a new '-F' flag to hmmc2 causing the hit alignment to be dumped to stdout as a stockholm file.

I can't reproduce the EBI setup so haven't yet been able to verify that hit metadata are propagated correctly, though it appears that sequence IDs (numeric ones) are certainly appearing in the output.

… HMM search - TODO: ensure sequence metadata is propagated
@npcarter
Copy link
Member

npcarter commented Mar 28, 2022 via email

@biomadeira
Copy link

Hi @npcarter! Just for a bit of context, this is related to the EBI tools framework (Job Dispatcher), which is not run by the HMMER team. In any case, we are running HMMER daemons and retrieving results with hmmc2 as instructed by them a few years back. Do you provide any "official" client to retrieve results or is it up to us to develop one?

@npcarter
Copy link
Member

npcarter commented Mar 29, 2022 via email

@biomadeira
Copy link

That helps! People come and go and I was not aware of some of the details. Yes, we are running our own HMMER daemons but were never intended to replace and/or compete with the HMMER team at EBI for this. Our collaboration with the HMMER team has been mostly down to the institute's requirement for having these tools presented and available via common APIs, which we (try to) provide.

Good to know Nicolo is working on a new iteration of the web tools. I will try to keep up with that and get in touch with him.

In relation to the PR, I can confirm Jim's changes compile and work as intended. As the scope of this feature is limited by the new additional flag I don't see any harm being caused by merging this in, but understand if you do not want to do it.

@foreveremain
Copy link
Author

foreveremain commented Oct 11, 2022 via email

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.

None yet

3 participants