Skip to content

Use default dependency key for RapidsMPF - #761

Merged
rapids-bot[bot] merged 3 commits into
rapidsai:mainfrom
bdice:use-default-rapidsmpf-dependency-key
Sep 3, 2026
Merged

Use default dependency key for RapidsMPF#761
rapids-bot[bot] merged 3 commits into
rapidsai:mainfrom
bdice:use-default-rapidsmpf-dependency-key

Conversation

@bdice

@bdicebdice commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Remove RapidsMPF's dependency_keys override so rapids-build-utils uses the default all key.

ray-default>=2.55.1 now resolves with PyTorch on both linux-64 and linux-aarch64, so the workaround introduced for rapidsai/rapidsmpf#906 is no longer needed.

Signed-off-by: Bradley Dice <bdice@bradleydice.com>
@bdice
bdice requested a review from a team as a code ownerSeptember 1, 2026 21:43
@bdice
bdice requested review from KyleFromNVIDIA and removed request for a teamSeptember 1, 2026 21:43

@trxcllnttrxcllnt 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.

Needs feature version bump, otherwise looks good

Signed-off-by: Bradley Dice <bdice@bradleydice.com>
@bdice
bdice requested a review from trxcllntSeptember 3, 2026 21:07
Signed-off-by: Bradley Dice <bdice@bradleydice.com>
@bdice

bdice commented Sep 3, 2026

Copy link
Copy Markdown
ContributorAuthor

/merge

@rapids-bot
rapids-botBot merged commit 2d16297 into rapidsai:mainSep 3, 2026
65 checks passed
rapids-botBot pushed a commit to rapidsai/rapidsmpf that referenced this pull request Sep 3, 2026
Remove the redundant `devcontainers` dependency file list. Development containers can use the default `all` key now that the Ray and PyTorch dependencies resolve together.
Depends on rapidsai/devcontainers#761.
Authors:
- Bradley Dice (https://github.com/bdice)
Approvers:
- Gil Forsyth (https://github.com/gforsyth)
URL: #1192
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@bdice@trxcllnt