search

desihub / desiconda

Tools to install DESI dependencies from scratch.
BSD 3-Clause "New" or "Revised" License
1 stars 7 forks source link

KPNO install fix #60

Closed sybenzvi closed 8 months ago

sybenzvi commented 1 year ago

A few changes to enable installation of desiconda packages on desi-7, intended to address the problems raised in #59:

marcelo-alvarez commented 1 year ago

@sybenzvi thanks for doing this, it looks great.

Removing specex and QuasarNP should be fine in the bootstrap — once we get specex finding the right libraries, we can put it back in and install it manually as opposed to in the bootstrap, with QuasarNP on a need-to-use basis.

Let's wait until you have a desiconda that is working in practice (which I assume you don’t quite have yet?). For example I’m surprised there are no settings you need to set in a conf/kpno-env.sh (which doesn’t exist but would be analogous to conf/nersc-env.sh).

sybenzvi commented 1 year ago

Let's wait until you have a desiconda that is working in practice (which I assume you don’t quite have yet?). For example I’m surprised there are no settings you need to set in a conf/kpno-env.sh (which doesn’t exist but would be analogous to conf/nersc-env.sh).

Good point, there are a few things that should be set (for example, there is no desi group on desi-7).

marcelo-alvarez commented 1 year ago

@sybenzvi checking in -- please let me know if I can help in any way with this. If I recall correctly, the plan is to use this branch to install a working desiconda at KPNO and, once that works, to review and merge the changes into main (mostly to ensure that it doesn't break desiconda at Perlmutter or make future changes more difficult than necessary, I think). @sbailey please let us know if you have any input about this. Thanks.

sbailey commented 8 months ago

@sybenzvi heads up: I'm going to merge this now so that we can use the mamba stuff for NERSC installs too. The rest is isolated to KPNO specific pieces. If those need further updates to work at KPNO, please open a new PR with that.

sybenzvi commented 8 months ago

Thanks @sbailey, any further updates needed for KPNO will be in a fresh PR.