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
Cython3 upgrade #1636
Cython3 upgrade #1636
Conversation
Note that the version of cython installed by apt is still 0.29.x |
Pull Request Test Coverage Report for Build 7531607762
💛 - Coveralls |
335ccc7
to
a655a4d
Compare
a655a4d
to
9ea2a51
Compare
A ton of warnings still from the cython compiler. Looking into resolutions... |
44f7ca2
to
77b236e
Compare
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.
Just one minor thing in the CHANGELOG
makes me wonder if we need a rebase?
d38c58d
to
4aacd52
Compare
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.
Thanks for moving us into the Cython 3 era @bennibbelink
This PR removes the pin on cython in cyclus, and we now see the same behavior with cython3.0.6 as we did in 0.29.x. Unfortunately it did not magically solve the issues discussed in #1589, so there is still some work to be done there
This has been branched off of
cmake-updates
, so I propose we resolve any needed revisions in #1632 and get that merged first to make this PR more readable