Skip to content

Remove raft-dask dependency#475

Merged
rapids-bot[bot] merged 1 commit intoNVIDIA:branch-25.10from
rgsl888prabhu:remove_raft_dask_deps
Oct 9, 2025
Merged

Remove raft-dask dependency#475
rapids-bot[bot] merged 1 commit intoNVIDIA:branch-25.10from
rgsl888prabhu:remove_raft_dask_deps

Conversation

@rgsl888prabhu
Copy link
Copy Markdown
Collaborator

Description

Remove unused dependency raft-dask

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

@rgsl888prabhu rgsl888prabhu added this to the 25.10 milestone Oct 9, 2025
@rgsl888prabhu rgsl888prabhu self-assigned this Oct 9, 2025
@rgsl888prabhu rgsl888prabhu requested review from a team as code owners October 9, 2025 16:09
@rgsl888prabhu rgsl888prabhu requested a review from AyodeAwe October 9, 2025 16:09
@rgsl888prabhu rgsl888prabhu added non-breaking Introduces a non-breaking change improvement Improves an existing functionality labels Oct 9, 2025
@rgsl888prabhu rgsl888prabhu requested a review from tmckayus October 9, 2025 16:09
Copy link
Copy Markdown
Member

@Iroy30 Iroy30 left a comment

Choose a reason for hiding this comment

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

Waiting for CI . If it passes, looks good to merge

@rgsl888prabhu
Copy link
Copy Markdown
Collaborator Author

@AyodeAwe May I get your review on this PR

@rgsl888prabhu
Copy link
Copy Markdown
Collaborator Author

/merge

@rapids-bot rapids-bot bot merged commit 831a2c7 into NVIDIA:branch-25.10 Oct 9, 2025
173 of 176 checks passed
rapids-bot bot pushed a commit that referenced this pull request Jan 5, 2026
…endencies.yaml (#744)

Follow up to #723

Makes some other changes to packaging configuration to reduce duplication and remove unused dependencies.

* removes all remaining Dask / `raft-dask` dependencies and configuration (missed in #475)
* uses more YAML anchors and `depends_on_{package}` groups in `dependencies.yaml` to reduce duplication

## Issue

N/A

## Notes for Reviewers

### How I found these Dask changes

Noticed `rapids-dask-dependency` in `dependencies.yaml` and I was surprised, as I thought cuOpt did not use Dask for anything.

Did this to find all the references and confirm there were no Dask-related direct imports in Python code here:

```shell
git grep -i dask
```

Then I searched the history and found #475. Noticed it had not updated the conda recipes or some of this other configuration.

#

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

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

URL: #744
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.

3 participants