Skip to content
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

cuproj: remove pin on 'proj' in conda packages #1449

Merged
merged 2 commits into from
Aug 30, 2024

Conversation

jameslamb
Copy link
Member

@jameslamb jameslamb commented Aug 29, 2024

Description

Proposes removing the cuproj conda package's pin on proj, so that it can build using the same version that conda-forge pins to in https://github.com/conda-forge/conda-forge-pinning-feedstock.

For context, see:

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@jameslamb jameslamb added 2 - In Progress Currenty a work in progress improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Aug 29, 2024
@github-actions github-actions bot added the conda Related to conda and conda configuration label Aug 29, 2024
@jameslamb jameslamb changed the title WIP: [NOT READY FOR REVIEW] cuproj: remove pin on 'proj' in conda packages cuproj: remove pin on 'proj' in conda packages Aug 29, 2024
@jameslamb jameslamb added 3 - Ready for Review Ready for review by team and removed 2 - In Progress Currenty a work in progress labels Aug 29, 2024
@jameslamb jameslamb marked this pull request as ready for review August 29, 2024 17:52
@jameslamb jameslamb requested a review from a team as a code owner August 29, 2024 17:52
Copy link
Member

@jakirkham jakirkham left a comment

Choose a reason for hiding this comment

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

Thanks James! 🙏

Yeah think this gets pinned periodically to workaround incomplete migrations. Am ok dropping

@jakirkham
Copy link
Member

/merge

@rapids-bot rapids-bot bot merged commit 1544e7b into rapidsai:branch-24.10 Aug 30, 2024
59 checks passed
@jameslamb jameslamb deleted the conda-proj-pin branch August 30, 2024 13:56
rapids-bot bot pushed a commit that referenced this pull request Sep 9, 2024
…as>=1.0 (#1453)

Contributes to rapidsai/build-planning#40

This PR adds support for Python 3.12.

## Notes for Reviewers

This is part of ongoing work to add Python 3.12 support across RAPIDS.
It temporarily introduces a build/test matrix including Python 3.12, from rapidsai/shared-workflows#213.

A follow-up PR will revert back to pointing at the `branch-24.10` branch of `shared-workflows` once all
RAPIDS repos have added Python 3.12 support.

Other changes required to add that support:

* pinning `proj` back to 9.3.x, reverting #1449 (#1453 (comment))
* requiring `geopandas >= 1.0` (#1453 (comment))

### This will fail until all dependencies have been updates to Python 3.12

CI here is expected to fail until all of this project's upstream dependencies support Python 3.12.

This can be merged whenever all CI jobs are passing.

Authors:
  - James Lamb (https://github.com/jameslamb)
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - Bradley Dice (https://github.com/bdice)

URL: #1453
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 - Ready for Review Ready for review by team conda Related to conda and conda configuration improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

2 participants