Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

KPNO install fix #60

Merged
merged 9 commits into from
Mar 22, 2024
Merged

KPNO install fix #60

merged 9 commits into from
Mar 22, 2024

Conversation

sybenzvi
Copy link

@sybenzvi sybenzvi commented Mar 7, 2023

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

  • Disable pip installation of hpsspy and mpi4py.
  • Disable installation of specex (requires BLAS libraries not installed) and QuasarNP.
  • Enable installation of nightwatch as a module.

@marcelo-alvarez
Copy link
Contributor

@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 sybenzvi marked this pull request as draft March 7, 2023 17:51
@sybenzvi
Copy link
Author

sybenzvi commented Mar 7, 2023

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
Copy link
Contributor

@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
Copy link
Contributor

sbailey commented Mar 22, 2024

@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.

@sbailey sbailey marked this pull request as ready for review March 22, 2024 18:15
@sbailey sbailey merged commit 3c58a02 into main Mar 22, 2024
@sbailey sbailey deleted the kpno-install-fix branch March 22, 2024 18:15
@sybenzvi
Copy link
Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants