Repository navigation
Conversation
gbarter
reviewed
Sep 18, 2026
| cat environment_ci.yml | ||
|
|
||
| - name: Install conda/mamba | ||
| uses: conda-incubator/setup-miniconda@v3 |
Member
There was a problem hiding this comment.
Looks like this action has a v4 available, should probably update that too
Build a single environment file from environment.yml plus the CI extras so conda solves once. Installing petsc4py/mpi4py/pyoptsparse into an already-solved environment sent the solver backtracking for 80+ minutes; the combined solve takes about 30 seconds. Pin openmpi so the MPI stack doesn't vary per Python version, and drop the apt mpich/libglu install, which was shadowed by conda's mpiexec anyway. Hold netcdf4 at 1.7.3. Builds 110+ of 1.7.4 are compiled with Cython 3.3.0 against the limited API and register the same shared Cython type module as petsc4py with an incompatible layout, so netCDF4 fails to import under mpiexec with "Shared Cython type cython_function_or_method has the wrong size". 1.7.3 is the last version with no limited-API builds. Also bump setup-miniconda to v4, add a job timeout, set PYTHONUNBUFFERED so signal-killed steps don't lose their output, and fix the coveralls condition that referenced Python 3.11 and could never match the matrix. Update README.md and docs/installation.rst to list the same packages. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
The conda solve for ubuntu-specific packages took ~1hr in the last PR. Solving in one shot can speed this up.
Type of change
What types of change is it?
Select the appropriate type(s) that describe this PR
Testing
Explain the steps needed to test the new code to verify that it does indeed address the issue and produce the expected behavior.
Checklist
Put an
xin the boxes that apply.