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

build(python): Officially support Python 3.13 #20549

Merged
merged 5 commits into from
Jan 7, 2025
Merged

build(python): Officially support Python 3.13 #20549

merged 5 commits into from
Jan 7, 2025

Conversation

stinodego
Copy link
Contributor

@stinodego stinodego commented Jan 3, 2025

Polars was already installable under Python 3.13 and already worked. This PR officially marks Python 3.13 as supported and updates our CI to also run tests under Python 3.13.

There's an issue with running MyPy with the latest NumPy version that came up because of this PR. I made a separate issue for this: #20561

@github-actions github-actions bot added internal An internal refactor or improvement python Related to Python Polars rust Related to Rust Polars labels Jan 3, 2025
Copy link
Contributor

github-actions bot commented Jan 5, 2025

The previous uncompressed lib size was Unknown MB.
The current uncompressed lib size after this PR is 1120.1477 MB.

Copy link

codecov bot commented Jan 5, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.98%. Comparing base (ca36b66) to head (5e2ceeb).
Report is 74 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #20549      +/-   ##
==========================================
- Coverage   79.05%   78.98%   -0.08%     
==========================================
  Files        1564     1564              
  Lines      220627   220759     +132     
  Branches     2502     2504       +2     
==========================================
- Hits       174413   174356      -57     
- Misses      45640    45829     +189     
  Partials      574      574              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@stinodego stinodego changed the title ci: Run CI for Python 3.13 ci(python): Run CI for Python 3.13 Jan 5, 2025
@stinodego stinodego removed the rust Related to Rust Polars label Jan 5, 2025
uv pip install -U typing-extensions
uv pip install --compile-bytecode -r requirements-dev.txt -r requirements-ci.txt --verbose
uv pip install -U typing-extensions
uv pip install --compile-bytecode -r requirements-dev.txt -r requirements-ci.txt --verbose --index-strategy=unsafe-best-match
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The --extra-index-url https://download.pytorch.org/whl/cpu entry in requirements-ci.txt caueses issues for NumPy installation under Python 3.13. Specifying this index strategy makes it so that NumPy is downloaded from the regular PyPI registry rather than the PyTorch registry.

@stinodego stinodego changed the title ci(python): Run CI for Python 3.13 feat(python): Officially support Python 3.13 Jan 5, 2025
@github-actions github-actions bot added the enhancement New feature or an improvement of an existing feature label Jan 5, 2025
@stinodego stinodego removed the internal An internal refactor or improvement label Jan 5, 2025
@stinodego stinodego marked this pull request as ready for review January 5, 2025 18:21
@stinodego stinodego changed the title feat(python): Officially support Python 3.13 build(python): Officially support Python 3.13 Jan 5, 2025
@stinodego stinodego removed the enhancement New feature or an improvement of an existing feature label Jan 5, 2025
@github-actions github-actions bot added the build Changes that affect the build system or external dependencies label Jan 5, 2025
@stinodego stinodego force-pushed the py313 branch 2 times, most recently from 61fcb16 to 7ed545f Compare January 5, 2025 20:53
@ritchie46 ritchie46 merged commit a7c6664 into main Jan 7, 2025
21 checks passed
@ritchie46 ritchie46 deleted the py313 branch January 7, 2025 13:39
@c-peters c-peters added the accepted Ready for implementation label Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepted Ready for implementation build Changes that affect the build system or external dependencies python Related to Python Polars
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants