desihub / desitarget

DESI Targeting
BSD 3-Clause "New" or "Revised" License
18 stars 23 forks source link

add SKYBRICKS_DIR and make module file more portable using DESI_ROOT #732

Closed sbailey closed 3 years ago

sbailey commented 3 years ago

@geordie666 this PR updates the etc/desitarget.module file to

You can inspect the environment that would be installed with this at NERSC via:

source /global/cfs/cdirs/desi/software/desi_environment.sh 21.3
module unload desitarget
module load desitarget/modulefile

(the last two lines are the same as a module swap desitarget/modulefile, but allows you to inspect the environment in between to ensure that it really is unsetting CMX_DIR etc. before resetting correctly with the new environment file.

I'd like to tag this as 1.0.1 to get the $SKYBRICKS_DIR update.

geordie666 commented 3 years ago

@sbailey: I tried out the 21.3 environment, and all of the directories behave exactly as I would expect. Feel free to merge this and tag 1.0.1 whenever you get a chance.

Don't forget to update the changes docs!