Skip to content

Conversation

@EliusSolis
Copy link
Contributor

Fixes to reflect changes in cython 3.11 and 3.12, mainly PEP 674 and python/cpython#101292

@Herbrant
Copy link

Hi! I tested your PR. I think you also need to replace the PyDict_GET_SIZE calls with PyUnicode_GET_LENGTH, then it works fine.
Thank you for your help.

Copy link
Contributor

@BatchClayderman BatchClayderman left a comment

Choose a reason for hiding this comment

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

Thank you for the nice update. I used this version to build the Python charm for Python 3.12 and it worked fine. I have been using this version with Python 3.12 under the Ubuntu (24.04.1 LTS) operating system (WSL) for over a month. Everything seems pretty. Testing schemes work correctly. It is my sincere hope that this branch can be merged. Merry Christmas and Happy New Year.

Copy link
Member

@jakinyele jakinyele left a comment

Choose a reason for hiding this comment

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

LGTM!

@jakinyele jakinyele merged commit 0661115 into JHUISI:dev Jan 23, 2025
@BatchClayderman
Copy link
Contributor

Hi! I tested your PR. I think you also need to replace the PyDict_GET_SIZE calls with PyUnicode_GET_LENGTH, then it works fine. Thank you for your help.

Hi, guys. Is what you stated before the same as what is described in #328 and #326 ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants