Thanks to visit codestin.com
Credit goes to github.com

Skip to content

CI Add Python 3.12 build #26886

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

Closed
wants to merge 7 commits into from
Closed

Conversation

lesteve
Copy link
Member

@lesteve lesteve commented Jul 24, 2023

Numpy and Scipy dev now have wheels for Python 3.12, this is a good opportunity to test scikit-learn with Python 3.12.

Locally the test suite run without errors with Python 3.12, let's see what the

We could also build and upload wheels to scientific-python-nightly-wheels, but maybe in a separate PR? Here is the Scipy PR that added wheels upload for Python 3.12 to draw inspiration from: scipy/scipy#18937.

@github-actions
Copy link

github-actions bot commented Jul 24, 2023

✔️ Linting Passed

All linting checks passed. Your pull request is in excellent shape! ☀️

Generated for commit: 6a8d8e0. Link to the linter CI: here

@lesteve lesteve marked this pull request as draft July 24, 2023 09:42
@lesteve lesteve marked this pull request as ready for review July 24, 2023 12:28
@lesteve
Copy link
Member Author

lesteve commented Jul 24, 2023

This is ready to review, the Python 3.12 build passes, the error is a HTTP 503 during codecov upload, so not related to this PR, see this build

[2023-07-24T12:39:30.122Z] ['error'] There was an error running the uploader: Error uploading to https://codecov.io/: Error: There was an error fetching the storage URL during POST: 503 - upstream connect error or disconnect/reset before headers. reset reason: connection failure
##[error]Bash exited with code '255'.

Copy link
Member

@thomasjpfan thomasjpfan left a comment

Choose a reason for hiding this comment

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

For testing Python dev versions, I prefer to add it to the wheel build directly and upload it to the nightly index.

I am -0.25 on adding a Python dev job to Azure, because the job is temporary. I see the Python 3.12 job getting removed when 3.12 comes out because one of the other jobs will start using it.

@lesteve
Copy link
Member Author

lesteve commented Jul 24, 2023

OK, makes sense.

I guess one day, a Python dev "permanent" build may make sense as suggested in https://discuss.scientific-python.org/t/spec-4-nightly-wheels/474/3. For now making sure that we are ready for Python 3.12 is higher priority.

@lesteve lesteve marked this pull request as draft August 7, 2023 12:11
@lesteve lesteve closed this Sep 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants