Upgraded cython to 3.0.x #1221
Merged
Upgraded cython to 3.0.x #1221
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
|
Cross-reference: assuming this works out this should address PYTHON-1390 as well. |
absurdfarce
approved these changes
Sep 23, 2024
Contributor
|
I'm going to merge this now in order to get this ball rolling. I think we'll have some work to do based on some of the changes that come in with Cython 3.0.x, specifically around the NumPy integration. Jenkins runs for this PR are showing something that looks very much like the problems described by PYTHON-1347 but in this case I believe they're caused by the changes in the interaction between Cython 3.0.x and NumPy. This needs to be investigated a bit more but it's not an argument for holding off on this upgrade; it seems pretty obvious that supporting Cython 3.0.x is the way forward for the Python driver. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Upgrade cython to 3.x which builds for Python 3.x by default.