Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

MNT: Enable wheels for Python 3.12#26582

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
ksunden merged 1 commit intomatplotlib:mainfromQuLogic:py312-wheel
Aug 30, 2023

Conversation

QuLogic
Copy link
Member

PR summary

We should probably figure out tests#24711, but that shouldn't stop us from making the wheels at least.

Fixes#26557

PR checklist

jarrodmillman reacted with heart emojijarrodmillman reacted with rocket emoji
@QuLogicQuLogic added Maintenance CI: Run cibuildwheelRun wheel building tests on a PR labelsAug 23, 2023
@QuLogicQuLogic added this to thev3.7.3 milestoneAug 23, 2023
@QuLogicQuLogicforce-pushed thepy312-wheel branch 2 times, most recently from699182c toc7d91b0CompareAugust 23, 2023 20:33
@effigies
Copy link

FWIW, I have had some successusing therequires table to pin specific versions for building:

[build-system]requires = ["setuptools","setuptools_scm[toml]>=6.2","cython",# Newer than NEP29-minimum: compile against oldest numpy available"numpy==1.26b1; python_version >= '3.12rc1'","numpy==1.24; python_version >= '3.11' and python_version < '3.12.dev0'",# NEP29-minimum as of Aug 17, 2023"numpy==1.22; python_version >= '3.8' and python_version < '3.11'",]build-backend ="setuptools.build_meta"

@QuLogicQuLogicforce-pushed thepy312-wheel branch 2 times, most recently fromf6cf7ad toc7b62daCompareAugust 23, 2023 20:48
@QuLogic
Copy link
MemberAuthor

FWIW, I have had some successusing therequires table to pin specific versions for building:

Yes, I did see that elsewhere, though I was hoping to avoid it since it's mostly a wheel-building issue.

# Newer than NEP29-minimum: compile against oldest numpy available"numpy==1.26b1; python_version >= '3.12rc1'","numpy==1.24; python_version >= '3.11' and python_version < '3.12.dev0'",# NEP29-minimum as of Aug 17, 2023"numpy==1.22; python_version >= '3.8' and python_version < '3.11'",

Note though that sinceNumPy 1.25, we no longer use NEP29 as the basis for thebuild requirement.

effigies reacted with thumbs up emoji

@effigies
Copy link

Note though that sinceNumPy 1.25, we no longer use NEP29 as the basis for thebuild requirement.

I just saw that (via#26443), and look forward to removing the extra lines from my projects.

@QuLogic
Copy link
MemberAuthor

Ah, no wheels for contourpy; I really don't want to have to figure out how to get full Meson support in there too; so waiting oncontourpy/contourpy#262

CIBW_ARCHS: ${{ matrix.cibw_archs }}
# Remove this once NumPy with Python 3.12 wheels is not pre-release.
CIBW_BEFORE_BUILD: >-
pip install certifi "pybind11>=2.6" "setuptools>=42" "setuptools_scm>=7" &&

Choose a reason for hiding this comment

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

Would you be willing to try adding contourpy'sbuild-requires? They're not likely to put out a wheel until numpy puts out a final 1.26.0 release, which is not going to happen until 3.12 final.

Suggested change
pip install certifi "pybind11>=2.6" "setuptools>=42" "setuptools_scm>=7" &&
pip install certifi "setuptools>=42" "setuptools_scm>=7" "meson >= 1.2.0" "meson-python >= 0.13.1" "pybind11 >= 2.10.4"

Copy link
Contributor

@neutrinocerosneutrinocerosAug 26, 2023
edited
Loading

Choose a reason for hiding this comment

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

Where are you getting this from ? CPython 3.12.0 is scheduled for early October while numpy 1.26 's milestone is currently aimed at September 1st.

Choose a reason for hiding this comment

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

Possibly a faulty memory. Apologies.

Copy link
Contributor

Choose a reason for hiding this comment

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

no problem, just asking :)

@QuLogicQuLogicforce-pushed thepy312-wheel branch 3 times, most recently from8899da7 to17ed86bCompareAugust 26, 2023 08:59
@QuLogicQuLogic marked this pull request as ready for reviewAugust 26, 2023 20:36
@ksundenksunden merged commit9dfa263 intomatplotlib:mainAug 30, 2023
@lumberbot-app
Copy link

Owee, I'm MrMeeseeks, Look at me.

There seem to be a conflict, please backport manually. Here are approximate instructions:

  1. Checkout backport branch and update it.
git checkout v3.7.xgit pull
  1. Cherry pick the first parent branch of the this PR on top of the older branch:
git cherry-pick -x -m1 9dfa263cfcd0ed46208c56aca16f3bc460bb9294
  1. You will likely have some merge/cherry-pick conflict here, fix them and commit:
git commit -am 'Backport PR #26582: MNT: Enable wheels for Python 3.12'
  1. Push to a named branch:
git push YOURFORK v3.7.x:auto-backport-of-pr-26582-on-v3.7.x
  1. Create a PR against branch v3.7.x, I would have named this PR:

"Backport PR#26582 on branch v3.7.x (MNT: Enable wheels for Python 3.12)"

And apply the correct labels and milestones.

Congratulations — you did some good work! Hopefully your backport PR will be tested by the continuous integration and merged soon!

Remember to remove theStill Needs Manual Backport label once the PR gets merged.

If these instructions are inaccurate, feel free tosuggest an improvement.

QuLogic added a commit that referenced this pull requestAug 30, 2023
…582-on-v3.8.xBackport PR#26582 on branch v3.8.x (MNT: Enable wheels for Python 3.12)
@ksunden
Copy link
Member

The backport was included in#26672

pllim reacted with hooray emoji

@ksundenksunden mentioned this pull requestSep 15, 2023
5 tasks
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@effigieseffigieseffigies left review comments

@neutrinocerosneutrinocerosneutrinoceros left review comments

@ksundenksundenksunden approved these changes

@oscargusoscargusoscargus approved these changes

Assignees
No one assigned
Labels
CI: Run cibuildwheelRun wheel building tests on a PRMaintenance
Projects
None yet
Milestone
v3.7.3
Development

Successfully merging this pull request may close these issues.

[ENH]: Nightly Python 3.12 builds
5 participants
@QuLogic@effigies@ksunden@oscargus@neutrinoceros

[8]ページ先頭

©2009-2025 Movatter.jp