Add ability to set library path via COMPLEASM_LIBRARY_PATH if --library_path unspecified #32
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adds the ability to specify the library path (what is set via
--library_path
) using the environment variableCOMPLEASM_LIBRARY_PATH
. The logic is:--library_path
options wherever they appear toNone
args.library_path == None
, then check if environment variableCOMPLEASM_LIBRARY_PATH
is setargs.library_path
mb_downloads
, which will be in the current directoryThis also modifies the
__init__
logic inDownloader
to do the same.This change enables using a central location for lineage sets, useful for streamlining project-wide storage or, for example, for HPC clusters such as ours where we've already downloaded the lineage sets to the same system-wide location for both BUSCO and compleasm. These lineage sets do not often change, so enabling the use of a common location for them is not just feasible but recommended.