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

[DEV] test import fix for esmpy >= 8.4.0 #87

Merged
merged 75 commits into from
Aug 1, 2023

Conversation

xylar
Copy link
Contributor

@xylar xylar commented Aug 1, 2023

Checklist

LinaMuryanto and others added 30 commits August 4, 2020 12:15
…n_n_readthedocs

add 0001-fix_version-fix_readthedocs_build_failures.patch
…python39-0-2_h9b01fe

Rebuild for python39
This is similar to what was done in the previous merge for the
cdms2 package, but this time for the regrid2 package.
Add a patch to disable mpi4py with an env. var.
Add test to catch my stupidity
For `cdscan` to work when MPI is disabled, we need to be able
to call `setNetcdfUseParallelFlag(0)`.  Before this patch, this
call attempted to raise an error but there was a typo in the
error name (`CDSMError` instead of `CDMSError`), causing added
confusion.  We only want to raise an error if the code is
attempting to enable parallel NetCDF usage but MPI is disabled.
Patch to fix disabling `mpi4py` in `cdms2.dataset`
@conda-forge-webservices
Copy link

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@xylar
Copy link
Contributor Author

xylar commented Aug 1, 2023

@conda-forge-admin, please rerender

@xylar
Copy link
Contributor Author

xylar commented Aug 1, 2023

I'm doing a test build to see if this fixes E3SM-Project/e3sm_diags#711

@@ -1,13 +1,27 @@
{% set name = "cdms2" %}
{% set version = "3.1.5.rc3" %}
{% set version = "3.1.5rc4" %}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm just hijacking this release name for convenience.

- conda-forge cdat_dev
channel_sources:
- conda-forge/label/cdat_dev,conda-forge,defaults
- conda-forge cdms2_dev
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm changing the dev label for convenience -- I don't want to bring in all packages from the cdat_dev label, just this one.

@xylar
Copy link
Contributor Author

xylar commented Aug 1, 2023

@conda-forge-admin, please restart CI

@xylar xylar merged commit 3d8431d into conda-forge:dev Aug 1, 2023
8 checks passed
@xylar xylar deleted the fix-esmpy-8.4.x branch August 1, 2023 09:05
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.

None yet

7 participants