-
Notifications
You must be signed in to change notification settings - Fork 45
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
Use Mamba to install CI build dependencies #2622
base: main
Are you sure you want to change the base?
Conversation
Use mamba in place of conda.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2622 +/- ##
==========================================
- Coverage 78.40% 78.34% -0.06%
==========================================
Files 610 611 +1
Lines 75322 75438 +116
Branches 10745 10772 +27
==========================================
+ Hits 59057 59104 +47
- Misses 14092 14150 +58
- Partials 2173 2184 +11 |
@phyy-nx, this was an attempt to get around the CI issue with running the
Now, using Mamba instead, we get a different error:
I can reproduce this more simply on Ubuntu 22.04 and RHEL8 when running
So it's a problem with installing Open MPI, as required by @phyy-nx, is Open MPI actually used in the |
How does it relate to the fact that the gemmi version is now unpinned and required to be >0.6.5, see #2618 ? |
It merely removes it from the complaint:
Gemmi being mentioned is a red herring, it’s a side-effect of the conda resolver giving unsatisfactory results (admittedly conda is much better now, than the hundreds of pages it used to dump out). |
Hi, mpi is used in lots of xfel_regression tests. dials/.azure-pipelines/unix-build.yml Line 114 in d706264
Could openmpi be dropped from this line? mpi4py will bring in either openmpi or mpich, depending on which satisfies the dependency chain, and either are fine for the xfel_regression tests. That would also sync up with how we do regular cctbx.xfel builds, where neither openmpi or mpich is specified: |
Sorry, I missed this when you first posted it. Let's give it a go. |
mpi4py will bring in either openmpi or mpich, depending on which satisfies the dependency chain, and either are fine for the xfel_regression tests. Co-authored-by: phyy-nx <asbrewster@lbl.gov>
85f1c41
to
2959053
Compare
Use Mamba in place of Conda, in the hope that it will allow us to set up the
xfel_regression
test environment without errors. At the moment, that stage of the Python 3.9 bootstrap CI is always failing.