Skip to content

DEP: Loosely pin Cython to 3.0 #56993

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

Merged
merged 1 commit into from
Jan 31, 2024

Conversation

rhshadrach
Copy link
Member

  • closes #xxxx (Replace xxxx with the GitHub issue number)
  • Tests added and passed if fixing a bug or adding a new feature
  • All code checks passed.
  • Added type annotations to new arguments/methods/functions.
  • Added an entry in the latest doc/source/whatsnew/vX.X.X.rst file if fixing a bug or adding a new feature.

@rhshadrach rhshadrach added the Dependencies Required and optional dependencies label Jan 21, 2024
@@ -8,7 +8,7 @@ dependencies:

# build dependencies
- versioneer[toml]
- cython=3.0.5
- cython~=3.0.5
Copy link
Member

Choose a reason for hiding this comment

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

Any reason for pinning to 3.0 instead of a more general >= 3.0 range?

Copy link
Member Author

Choose a reason for hiding this comment

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

I meant to put this in the OP but forgot: I brought this up at the dev meeting in December. I am wondering if minor releases should be benchmarked first before we upgrade. The general consensus was that patch versions are fine to automatically take, but there was uncertainty around minor versions.

No opposition to >= if people are comfortable with that.

Copy link
Member Author

@rhshadrach rhshadrach Jan 22, 2024

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

This means any 3.0.x? Neat.

Copy link
Member

Choose a reason for hiding this comment

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

OK cool. I have no objection to this, as long as someone takes the time to do it (your benchmarks were very helpful with the 3.x upgrade)

@mroeschke mroeschke added this to the 3.0 milestone Jan 31, 2024
Copy link
Member

@mroeschke mroeschke left a comment

Choose a reason for hiding this comment

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

Sure let's try this out. If we find patch version are causing CI troubles often then we can reconsider

@mroeschke mroeschke merged commit 1785fdc into pandas-dev:main Jan 31, 2024
@mroeschke
Copy link
Member

Thanks @rhshadrach

@mroeschke
Copy link
Member

Ah sorry should have looked at the failing check more closely, could you fix the pinning in the requirements-dev.txt? https://github.com/pandas-dev/pandas/actions/runs/7731187330/job/21078258557

@rhshadrach
Copy link
Member Author

Yea - just noticed it. On it.

@rhshadrach rhshadrach deleted the enh_dep_loose_pin_cython branch January 31, 2024 21:48
pmhatre1 pushed a commit to pmhatre1/pandas-pmhatre1 that referenced this pull request May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dependencies Required and optional dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants