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

@bdice bdice 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 owner September 1, 2026 21:43
@bdice
bdice requested review from KyleFromNVIDIA and removed request for a team September 1, 2026 21:43

@trxcllnt trxcllnt 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 trxcllnt September 3, 2026 21:07
Signed-off-by: Bradley Dice <bdice@bradleydice.com>
@bdice

bdice commented Sep 3, 2026

Copy link
Copy Markdown
Contributor Author

/merge

@rapids-bot
rapids-bot Bot merged commit 2d16297 into rapidsai:main Sep 3, 2026
65 checks passed
rapids-bot Bot 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 free to 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