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

Python 3.13 Support, 3.8 EOL #787

Merged
merged 1 commit into from
Jan 8, 2025
Merged

Conversation

ax3l
Copy link
Member

@ax3l ax3l commented Jan 8, 2025

Add support for Python 3.13.
Remove 3.8 because it is EOL as of Oct 2024.

Bump to pybind11 2.13.0+, which add Python 3.13 support in CI.

Same as:

Note that we remove Python 3.8 only from documentation for now, but do not hard enforce that it is not used (because old Ubuntu versions and CI still use it). Enforcement would go in CMakeLists.txt (find_package(Python 3.9 ...)) and in setup.py (python_requires=">=3.8").

Add support for Python 3.13.
Remove 3.8 because it is EOL as of Oct 2024.

Bump to pybind11 2.13.0+, which add Python 3.13 support in CI.
@ax3l ax3l added install component: third party Changes in ImpactX that reflect a change in a third-party library component: python Python bindings labels Jan 8, 2025
@ax3l ax3l requested review from cemitch99 and EZoni January 8, 2025 19:41
Copy link
Member

@EZoni EZoni Jan 8, 2025

Choose a reason for hiding this comment

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

Is there a reason why we have 2.13.0 in one line and v2.13.6 in another line or is it just a typo?

Copy link
Member Author

@ax3l ax3l Jan 8, 2025

Choose a reason for hiding this comment

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

The one line is the minimum requirement for find_package (any newer than 2.13.0 will work), the other is the exact version we download on the fly (latest patch level, because great).

Copy link
Member

@cemitch99 cemitch99 left a comment

Choose a reason for hiding this comment

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

This looks good to me.

@ax3l ax3l merged commit ff93276 into BLAST-ImpactX:development Jan 8, 2025
16 checks passed
@ax3l ax3l deleted the py-3.13 branch January 8, 2025 22:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: python Python bindings component: third party Changes in ImpactX that reflect a change in a third-party library install
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants