Skip to content

include Python constraints for pure-Python conda packages#51

Merged
rgsl888prabhu merged 1 commit intoNVIDIA:branch-25.05from
jameslamb:python-constraints
May 30, 2025
Merged

include Python constraints for pure-Python conda packages#51
rgsl888prabhu merged 1 commit intoNVIDIA:branch-25.05from
jameslamb:python-constraints

Conversation

@jameslamb
Copy link
Copy Markdown
Member

The cuopt-sh-client and cuopt-server packages are pure Python.

However, their conda packages are not technically built as noarch: python because of some historical limitations in RAPIDS build workflows (to be addresses as part of rapidsai/build-planning#43).

As a result, 1 of those packages is published for each combination of (Python minor version, CPU architecture) supported by RAPIDS. See, for example, https://anaconda.org/nvidia/cuopt-server/files?version=25.05.00

image

For as long as we're doing that, those packages should have a constraint on python, so that (for example) only 1 package matches for a given (Python minor version, CPU architecture).

This does that.

@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot bot commented May 30, 2025

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@rgsl888prabhu rgsl888prabhu added bug Something isn't working non-breaking Introduces a non-breaking change labels May 30, 2025
@rgsl888prabhu
Copy link
Copy Markdown
Collaborator

/ok to test

@jameslamb
Copy link
Copy Markdown
Member Author

I think this is working the way we want it to:

 │ │ Finalized run dependencies (cuopt-server-25.05.00a35-py311_250530_0b6e1a6b):
 │ │ ╭──────────────────┬───────────────────────────────────────╮
 │ │ │ Name             ┆ Spec                                  │
 │ │ ╞══════════════════╪═══════════════════════════════════════╡
 │ │ │ Run dependencies ┆                                       │
...
 │ │ │ python           ┆                                       │
...
 │ │ │ python_abi       ┆ 3.11.* *_cp311 (RE of [host: python]) │
 │ │ ╰──────────────────┴───────────────────────────────────────╯

(build link)

@jameslamb jameslamb marked this pull request as ready for review May 30, 2025 16:36
@jameslamb jameslamb requested a review from a team as a code owner May 30, 2025 16:36
@jameslamb jameslamb requested a review from rgsl888prabhu May 30, 2025 16:36
@jameslamb jameslamb changed the title WIP: include Python constraints for pure-Python conda packages include Python constraints for pure-Python conda packages May 30, 2025
Copy link
Copy Markdown
Contributor

@gforsyth gforsyth left a comment

Choose a reason for hiding this comment

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

🎉

@rgsl888prabhu
Copy link
Copy Markdown
Collaborator

/merge

@rgsl888prabhu rgsl888prabhu merged commit 63bf575 into NVIDIA:branch-25.05 May 30, 2025
58 checks passed
@jameslamb jameslamb deleted the python-constraints branch September 22, 2025 13:50
jieyibi pushed a commit to yining043/cuopt that referenced this pull request Mar 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants