-
Notifications
You must be signed in to change notification settings - Fork 6
Change Novoalign source to use conda package and separate novoalign.lic file #26
Comments
Thanks for the update @tomkinsc Clarifying, Novoalign requires a commercial license for multi-threaded operation (which is why the If that's the case, I'll plan to expose the |
Exactly, the paid license buys multi-threading, but the binary is the same for non-commercial or commercial use. Your plan sounds great, thanks! |
@tomkinsc, a couple of clarifying questions while I take a dig at this:
|
Hi,
|
Thanks Danny! |
Addressed by #30. Ideally, we want to test this using a license file to make sure that the wrapper does things correctly and Novoalign will be able to multi-thread when a license file is given. @tomkinsc or @dpark01 would you happen to have a license file that we can use to test? You can use any of the assembly workflows in this folder which would accept a |
Closing with confirmation with @tomkinsc |
Novoalign is available as a conda package, so we no longer need to unpack it as a custom tarball and set
NOVOALIGN_PATH
. The DNAnexus wrappers can omit settingNOVOALIGN_PATH
, which will cause viral-ngs to install the conda version of novoalign (or the conda package can be installed in advance). By default, the conda package runs in single-threaded mode until anovoalign.lic
file is provided in the same directory as the novoalign binary (in this case<conda-env>/bin
). Thenovoalign.lic
file from the old tarball can be reused, but we would ideally like to be able to specify it as an input for the applets.As of this commit the
novoalign.lic
file can be copied to the conda environment in two ways:NOVOALIGN_LICENSE_PATH
environment variable (viral-ngs will copy it in)novoalign-license-register
command (provided by the conda package, and available if the conda environment is active)The text was updated successfully, but these errors were encountered: