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

Query: 2 new files seen in source code. #32

Closed
sowani opened this issue Nov 22, 2016 · 2 comments
Closed

Query: 2 new files seen in source code. #32

sowani opened this issue Nov 22, 2016 · 2 comments

Comments

@sowani
Copy link

sowani commented Nov 22, 2016

Hi, I have been working on hhsuite source code for some time so my version of source code is slightly older (I think more than a month or so). I cloned hhsuite yesterday as I wanted to have a local reference copy. While comparing the files I found that there are 2 new files in the current codebase:
src/hhblits_mpi.cpp
lib/ffindex/src/ffindex_from_fasta_with_split.c

I did not see anything related to this in the CHANGES file either. The version strings (in top level CMakeLists.txt) are same in both code bases. Are these 2 new files essential for hhsuite or they are just kind of utilities which are not necessary for core functioning of hh-suite?

Thanks,
Atul.

@milot-mirdita
Copy link
Member

Hi Atul,

The two files build an utility executable "hhblits_mpi" that replaces the usual "ffindex_apply_mpi ... -- hhblits ..." calls, this lowers the overhead by quite a bit, when you start hhblits searches for many thousand small query sequences.

But they are not related to the core search functionality.

Best,
Milot

@sowani
Copy link
Author

sowani commented Nov 25, 2016

Thanks @milot-mirdita for the explanation. So I will defer checking these files for my current porting work. Once I am done with immediate high priority tasks (like fixing crashes in ported code etc.) I will have a look at these files.
I am closing this issue now.

Mit freundlichen Grüßen/Best regards,
Atul

@sowani sowani closed this as completed Nov 25, 2016
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

No branches or pull requests

2 participants