Skip to content

Skip cp314t for now and build cp314 on PRs#1415

Merged
jswhit merged 2 commits into
Unidata:masterfrom
ocefpaf:bump_cibuildwheel
Aug 6, 2025
Merged

Skip cp314t for now and build cp314 on PRs#1415
jswhit merged 2 commits into
Unidata:masterfrom
ocefpaf:bump_cibuildwheel

Conversation

@ocefpaf

@ocefpaf ocefpaf commented Aug 1, 2025

Copy link
Copy Markdown
Collaborator

@jswhit I also had to update many of the dependencies in https://github.com/ocefpaf/netcdf-manylinux and the manylinux version to get this working with Python 3.14. The build is working now, but 2 tests are when excuted with the built wheel. Do you mind taking a look? I'm not sure I know where to start fixing them. Maybe downgrading hdf5?

Closes #1413

@ocefpaf ocefpaf force-pushed the bump_cibuildwheel branch from fbd0584 to 12a5159 Compare August 1, 2025 19:31
@jswhit

jswhit commented Aug 2, 2025

Copy link
Copy Markdown
Collaborator

Not an HDF5 issue - it seems that numpy 2.3.2 is pickier about how it handles iterable slice indices. I will have a look.

@jswhit

jswhit commented Aug 2, 2025

Copy link
Copy Markdown
Collaborator

hmmm - not able to reproduce this on my mac with python 3.13.5 and numpy 2.3.2

@jswhit

jswhit commented Aug 3, 2025

Copy link
Copy Markdown
Collaborator

If you update your fork from master, the tests should now pass on python 3.14

@jswhit

jswhit commented Aug 4, 2025

Copy link
Copy Markdown
Collaborator

Looks like macos-12 is no longer supported, macos-13 is the only one that supports x86_64

@ocefpaf ocefpaf force-pushed the bump_cibuildwheel branch from ac2553d to c335f0d Compare August 4, 2025 10:54
@ocefpaf

ocefpaf commented Aug 4, 2025

Copy link
Copy Markdown
Collaborator Author

Looks like macos-12 is no longer supported, macos-13 is the only one that supports x86_64

Thanks! I updated that and it should be OK for a review now.

PS: To make our lives easier in the future, note that these Linux builds are using the Docker images from https://github.com/ocefpaf/netcdf-manylinux/tree/35e29a405d74de914bfb9427bf4883eb9d4ced26. I did update to use a stable release of netcdf-c in the next build. Hopefully there won't be any significant changes.

Also, we should really move that repo away from my personal account to make it easier for others to find and contribute.

@ocefpaf

ocefpaf commented Aug 5, 2025

Copy link
Copy Markdown
Collaborator Author

@jswhit do you think we should try cp314t? The free-threaded Python variant?

@jswhit

jswhit commented Aug 5, 2025

Copy link
Copy Markdown
Collaborator

I don't think the netcdf-c library is thread safe (there was some initial work done to fix this here, but it looks unfinished). Given this, maybe there is not much benefit?

@ocefpaf

ocefpaf commented Aug 5, 2025

Copy link
Copy Markdown
Collaborator Author

Given this, maybe there is not much benefit?

It is likely to fail too. I forgot about that.

@jswhit jswhit merged commit c3150e5 into Unidata:master Aug 6, 2025
39 checks passed
@ocefpaf ocefpaf deleted the bump_cibuildwheel branch August 6, 2025 15:46
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.

2 participants