Conversation
|
I've disabled the Cygwin workflow because it is failing the build with the new pyproject.toml. Cygwin plans to skip |
| CIRRUS_CLONE_SUBMODULES: true | ||
| CIBW_BUILD: cp39-* | ||
| EXPECT_CPU_FEATURES: NEON NEON_FP16 NEON_VFPV4 ASIMD ASIMDHP ASIMDDP ASIMDFHM | ||
| - env: |
There was a problem hiding this comment.
Do we want to move the EXPECT_CPU_FEATURES line to the 3.10 test?
There was a problem hiding this comment.
It wasn't clear to me when I was modifying the ci where these are actually used. It was left over from Travis ci configs
|
Not sure why, but it looks like the freebsd tests didn't trigger on this PR and did on #26243. Those tests will need to be updated because freebsd defaults to python3.9. |
Maybe |
Yeah, I think you need to explicitly install python3.10 then e.g. use a |
Looks like removing the [wheel build] fixes that, so there is a logic error somewhere. |
|
Should be good to go. |
mattip
left a comment
There was a problem hiding this comment.
Two small nits, otherwise LGTM
Drop Python 3.9 from the supported Python versions and CI. Python 3.9 is still used in the Cygwin build test because Cygwin does not yet support 3.10.
9a04db9 to
2452ff4
Compare
Source for Python update plans. It looks like CPython dropped some Cygwin-specific sections a few years back, and are aware the current build system is hard to change, so getting a new version of Python working on Cygwin might take longer than usual. In the meantime, I think I saw mentions of CI runs on BSD, which should keep Cygwin from breaking too badly. I think SIMD stack alignment issues would be the main difference from Linux they wouldn't notice. For the question from #26244, I suspect dropping Cygwin will make the other platforms simpler. I'll be back whenever a test version of 3.12 comes out to figure out how to get CI to use that (either cygwin/cygwin-install-action#12 or the version-specifications of egor-tensin/setup-python). |
following numpy and pymatgen numpy/numpy#26222 materialsproject/pymatgen#4009
* bump min supported python to 3.10 following numpy and pymatgen numpy/numpy#26222 materialsproject/pymatgen#4009 * pass strict=True to zip() added in python 3.10 safer, since it errors if iterables have different lengths * fix UP038 Use `X | Y` in `isinstance` call instead of `(X, Y)`
Drop Python 3.9 from the supported Python versions and CI.
Cygwin is excepted because it does not yet support 3.10.