-
Notifications
You must be signed in to change notification settings - Fork 15
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
Make Python 3.13 Compatible #79
Conversation
# Conflicts: # Cargo.lock # Cargo.toml
I was working on but if we can make the jump to pyo3 0.22 in this PR, I think that's preferable :) |
You need to add 3.13 to Lines 60 to 66 in 2fce424
|
@edgarrmondragon Thank you so much for the tips! I'll take a look at your PR tomorrow and can merge anything that can be added. I'm not quite confident in my implementation. If you have time, it would be great to hear your ideas. Thanks! |
Changes look good to me, but I'm also not super familiar with pyo3' API (old or new) 😅. Maybe @davidhewitt can take a quick look in case something's majorly fumbled here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Broadly this looks correct to me 👍
@Julian Hi Julian, this PR should be ready for review. It bumps PyO3 to 0.22.1 and passes tests in Python 3.13 with GIL. However, it still breaks under the free-threaded Python. I'm happy to wait for updates from PyO3 and integrate so that rpds works on both variants of 3.13. What do you think? |
PyO3 will likely be a release or two before we support free-threaded Python properly, though it's very high priority on my TODO list. I think probably we should be adjusting PyO3 to fail builds with a helpful error on free-threaded builds for the moment (maybe will release as 0.22.2). So I would advise not to wait for free-threaded support from us :) |
@davidhewitt Thank you so much for your insight! I can submit a PR to allow PyO3 fail builds on free-threaded Python if that's helpful. :) |
Thank you all for your help here, it's really appreciated! I'm going to merge I think. Thanks again! |
@FlickerSoul a PR would be greatly appreciated. You'd need to modify |
6 minutes ago I added 3.13 to my CI setup and rpds was failing. Looked up the issues, saw 3 minutes ago rpds v0.19.0 was released, re-ran and it passed. Awesome timing, and thanks a lot! |
This is currently a draft PR. See #72 (comment).