Add runHMMER function for HMMER analysis#18
Conversation
These functions will be added to data_processing.R once approved. The scripts are modifications of @epbrenner 's hmmering and rhmmer.
Removed redundant line reading from file as it was already handled earlier in the code.
jananiravi
left a comment
There was a problem hiding this comment.
It looks like you moved a line -- unless I'm missing something, it's good to merge!
jananiravi
left a comment
There was a problem hiding this comment.
I see that I commented on one commit earlier -- sorry about that!
In principle, it looks good. I would like to request @eboyer221 or @epbrenner to run this locally and suggest non-alpine placeholders to ensure this works for all!
|
Just had a thought, we have to run each HMMER database in the function separately and then combine the outputs later? |
Co-authored-by: Abhirupa Ghosh <100681585+AbhirupaGhosh@users.noreply.github.com>
|
From @charmvang
@AbhirupaGhosh, do we integrate these results and return a single df/parquet following the individual runs? 🤔 |
Confirmed with @AbhirupaGhosh that the multi-database aggregation is going to be a separate follow-up PR rather than part of this one. .runHMMER keeps a single database_path for now and users call it once per database. |
…rseHMMEROutput internal
These functions will be added to data_processing.R once approved.
The scripts are modifications of @epbrenner 's hmmering and rhmmer.
Description
What kind of change(s) are included?
Checklist
Please ensure that all boxes are checked before indicating that this pull request is ready for review.