Closed danejo3 closed 1 year ago
Test suite is at 100% coverage.
Ready for an initial review for this PR! @standage
I presume we can close this thread now that #16 has been merged?
I presume we can close this thread now that #16 has been merged?
Yup! No need for this anymore!
YEAT'd and DELETED!
YEAT'd and DELETED!
The purpose of this PR is to lay down the foundations of #10 .
In this PR, users will now be required to provide a
config
file that contains information on what assembly algorithms that are to be used.The reason why having this
config
file would be helpful is to allow users to run multiple assemblers in the future asynchronously. By having multiple assembly algorithms running at the sametime, this will allow the user to work with various assemblers based on time, resource, and accuracy constraints.This PR does not provide the parallel jobs support and additional assemblers such as
MEGAHIT
. Separate PR will be created for these upcoming features.