Skip to content

Conversation

@jameslamb
Copy link
Member

@jameslamb jameslamb commented Jan 5, 2026

Description

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 Remove raft-dask dependency #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:

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.

Checklist

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

Summary by CodeRabbit

Release Notes

  • Chores
    • Removed rapids-dask-dependency and raft-dask from project dependencies across multiple environment configurations, simplifying the dependency tree.
    • Updated copyright year to 2025-2026.
    • Adjusted import grouping configuration for improved import organization.

✏️ Tip: You can customize this high-level summary in your review settings.

@jameslamb jameslamb added non-breaking Introduces a non-breaking change improvement Improves an existing functionality labels Jan 5, 2026
@copy-pr-bot
Copy link

copy-pr-bot bot commented Jan 5, 2026

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

/ok to test

@jameslamb jameslamb changed the title WIP: remove dask dependencies and configuration, reduce duplication in dependencies.yaml remove dask dependencies and configuration, reduce duplication in dependencies.yaml Jan 5, 2026
@jameslamb jameslamb marked this pull request as ready for review January 5, 2026 14:25
@jameslamb jameslamb requested a review from a team as a code owner January 5, 2026 14:25
@jameslamb jameslamb requested a review from msarahan January 5, 2026 14:25
@coderabbitai

This comment was marked as spam.

@rgsl888prabhu
Copy link
Collaborator

/merge

@rapids-bot rapids-bot bot merged commit e9354c1 into NVIDIA:main Jan 5, 2026
182 of 183 checks passed
@jameslamb jameslamb deleted the dependencies-yaml branch January 5, 2026 17:59
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