Skip to content

include Python constraints for pure-Python conda packages - #51

Merged
ramakrishnap-nv merged 1 commit into
NVIDIA:branch-25.05from
jameslamb:python-constraints
May 30, 2025
Merged

include Python constraints for pure-Python conda packages#51
ramakrishnap-nv merged 1 commit into
NVIDIA: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-pr-bot Bot commented May 30, 2025

Copy link
Copy Markdown

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.

@ramakrishnap-nv ramakrishnap-nv added bug Something isn't working non-breaking Introduces a non-breaking change labels May 30, 2025
@ramakrishnap-nv

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 ramakrishnap-nv 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

@gforsyth gforsyth left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

🎉

@ramakrishnap-nv

Copy link
Copy Markdown
Collaborator

/merge

@ramakrishnap-nv
ramakrishnap-nv merged commit 63bf575 into NVIDIA:branch-25.05 May 30, 2025
@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