Skip to content

Conversation

@jameslamb
Copy link
Member

Description

Fixes two small packaging issues with cuopt-server:

  • removes unnecessary dependency on httpx (cuopt-server does not use this)
  • declares dependency on psutil (this is declared in wheels but was missing in conda packages)

Notes for Reviewers

How I found these

Was looking at the conda recipes and checked these things, similar to this:

git grep httpx

Checklist

  • I am familiar with the Contributing Guidelines.
  • Testing
    • New or existing tests cover these changes
    • Added tests
    • Created an issue to follow-up
    • NA
  • Documentation
    • The documentation is up to date with these changes
    • Added new documentation
    • NA

@jameslamb jameslamb added non-breaking Introduces a non-breaking change improvement Improves an existing functionality labels Sep 23, 2025
@copy-pr-bot
Copy link

copy-pr-bot bot commented Sep 23, 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.

@jameslamb
Copy link
Member Author

Skipping triggering CI until the RAPIDS-wide nightly issues are resolved: #408 (comment)

@jameslamb
Copy link
Member Author

/ok to test

@jameslamb
Copy link
Member Author

I think this is ready for review. The docs-build failures are a known issue and not related to this PR, described here: #408 (comment)

@jameslamb jameslamb changed the title WIP: cuopt-server: update dependencies (drop httpx, add psutil) cuopt-server: update dependencies (drop httpx, add psutil) Sep 24, 2025
@jameslamb jameslamb marked this pull request as ready for review September 24, 2025 03:29
@jameslamb jameslamb requested a review from a team as a code owner September 24, 2025 03:29
@jameslamb jameslamb requested a review from AyodeAwe September 24, 2025 03:29
@jameslamb jameslamb removed the request for review from AyodeAwe September 24, 2025 15:13
@jameslamb
Copy link
Member Author

/merge

@rapids-bot rapids-bot bot merged commit ac1c293 into NVIDIA:branch-25.10 Sep 24, 2025
172 of 174 checks passed
@jameslamb jameslamb deleted the remove-unused branch September 24, 2025 21:29
aliceb-nv pushed a commit that referenced this pull request Sep 26, 2025
Fixes two small packaging issues with `cuopt-server`:

* removes unnecessary dependency on `httpx` (`cuopt-server` does not use this)
* declares dependency on `psutil` (this is declared in wheels but was missing in conda packages)

## Notes for Reviewers

### How I found these

Was looking at the conda recipes and checked these things, similar to this:

```shell
git grep httpx
```

#

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

Approvers:
  - Ramakrishnap (https://github.com/rgsl888prabhu)

URL: #413
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants