simonsobs / soconda

Simons Observatory Conda Tools
BSD 2-Clause "Simplified" License
1 stars 2 forks source link

Testing deployment before tag #21

Closed tskisner closed 8 months ago

tskisner commented 9 months ago

Testing install across several systems. Updating github actions.

tskisner commented 9 months ago
tskisner commented 9 months ago

Another note of something I'll fix here: conda-build (the plugin) must now be installed in the base environment along with conda. There is further discussion on these two pages about why this is now needed, due to changes in the conda plugin system:

https://docs.conda.io/projects/conda-build/en/stable/install-conda-build.html https://github.com/conda/conda-build/issues/4995

Note that the compilers package remains installed in the non-base environment. It is only conda plugins (like conda-build) that must be in base.

tskisner commented 9 months ago

Hmmm, actually there are multiple problems, including issues with upstream dependencies, related to python-3.12. Will save those for a future PR.

Another note of something I'll fix here: conda-build (the plugin) must now be installed in the base environment along with conda. There is further discussion on these two pages about why this is now needed, due to changes in the conda plugin system:

https://docs.conda.io/projects/conda-build/en/stable/install-conda-build.html conda/conda-build#4995

Note that the compilers package remains installed in the non-base environment. It is only conda plugins (like conda-build) that must be in base.

tskisner commented 9 months ago

This is now ready for review:

Bai-Chiang commented 9 months ago

To redirect stderr and stdout to a file only, the old format is correct command >>log 2>&1 or bash specific style command &>>log. The reason of this ordering is explained here.

Also I will make some change for micromamba setup for conda-build changes, since its base env is empty.

tskisner commented 9 months ago

There are more changes coming to this PR (unfortunately). I am debugging segfaults with openmp dependencies (getting closer there), and also dealing with the consequences of conda build now running from the base environment. This impacts things like requiring that conda-build and conda-verify are installed in the base environment, and by default the conda-bld directory in the base environment is used (problematic since it may not be writable). Anyway, sorry for the delay, but wanted you to know I'm not done yet here

iparask commented 9 months ago

@tskisner, there is a new package in sotodlib, numdifftools. See here

tskisner commented 8 months ago

I believe all previous review comments are addressed. The remaining segfault from OpenMP when importing scipy before toast is actually a separate problem that began with the latest packaging of scipy-1.12. I would like to address that in a separate PR since this one has been hanging around too long already. @Bai-Chiang , any further changes needed here before merging?

Bai-Chiang commented 8 months ago

I will make few changes for micromamba, since conda-build is now installed in base environment, this currently breaks micromamba installation.

tskisner commented 8 months ago

Ok, sounds good. In the mean time I am going to setup a new install of this branch at NERSC for testing.