desihub / desiutil

General DESI utilities, shell scripts, desiInstall, etc.
BSD 3-Clause "New" or "Revised" License
3 stars 9 forks source link

add_dependencies includes envvars #183

Closed sbailey closed 2 years ago

sbailey commented 2 years ago

This PR addresses desihub/desispec#1077 by adding the following environment variables to the suite of dependencies added by desiutil.depend.add_dependencies:

this expands the concept of "dependencies" to be not just code versions but also input directories. This list can be overridden with envvar_names option analogous to pre-existing module_names option.

I will make a companion PR on desispec to make sure we call this for every desispec.io.write* function, but this PR doesn't need to wait for that.

@weaverba137 if you have a chance to take a look, please do, but I plan to merge later today when I get to the point of needing this for testing.