Skip to content

Remove UCX-Py#7130

Merged
rapids-bot[bot] merged 5 commits intorapidsai:branch-25.10from
pentschev:remove-ucx-py
Aug 26, 2025
Merged

Remove UCX-Py#7130
rapids-bot[bot] merged 5 commits intorapidsai:branch-25.10from
pentschev:remove-ucx-py

Conversation

@pentschev
Copy link
Copy Markdown
Member

UCX-Py is deprecated and being archived. This change removes all UCX-Py tests, leaving only UCXX tests, and all mentions in favor of UCXX. See rapidsai/ucx-py#1160 for details.

Copy link
Copy Markdown
Member

@jameslamb jameslamb left a comment

Choose a reason for hiding this comment

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

I searched like this:

git grep -E 'ucx\-py|ucp'

Found just one more use.

This old example (using Corey's fork of ucx-py 😅) should probably be updated / removed:

git clone https://github.com/cjnolet/ucx-py.git

Marking this "approve", so you all can do whatever you want with that and not need another review from me.

Comment thread python/cuml/tests/dask/test_dask_nearest_neighbors.py
@csadorf csadorf added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Aug 25, 2025
@pentschev
Copy link
Copy Markdown
Member Author

This old example (using Corey's fork of ucx-py 😅) should probably be updated / removed:

I'll leave this up to cuML folks, the example hasn't been updated since 2023 apparently, and is still pointing to CUDA 9.2! I think it's safe to say this is not being actively used.

@csadorf
Copy link
Copy Markdown
Contributor

csadorf commented Aug 25, 2025

This old example (using Corey's fork of ucx-py 😅) should probably be updated / removed:

I'll leave this up to cuML folks, the example hasn't been updated since 2023 apparently, and is still pointing to CUDA 9.2! I think it's safe to say this is not being actively used.

I am not sure whether the example is still relevant in its current form. However, unless we can verify that the instructions still work work with ucxx, we should just mark them as deprecated by adding a deprecation notice at the top of the page.

@pentschev
Copy link
Copy Markdown
Member Author

This old example (using Corey's fork of ucx-py 😅) should probably be updated / removed:

I'll leave this up to cuML folks, the example hasn't been updated since 2023 apparently, and is still pointing to CUDA 9.2! I think it's safe to say this is not being actively used.

I am not sure whether the example is still relevant in its current form. However, unless we can verify that the instructions still work work with ucxx, we should just mark them as deprecated by adding a deprecation notice at the top of the page.

Added a deprecation notice in 6e1b508 .

Copy link
Copy Markdown
Contributor

@csadorf csadorf left a comment

Choose a reason for hiding this comment

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

LGTM, thanks a lot!

Comment thread wiki/mnmg/Using_Infiniband_for_MNMG.md Outdated
@csadorf
Copy link
Copy Markdown
Contributor

csadorf commented Aug 26, 2025

/merge

@rapids-bot rapids-bot Bot merged commit b09f35f into rapidsai:branch-25.10 Aug 26, 2025
76 checks passed
@pentschev
Copy link
Copy Markdown
Member Author

Thanks all for the reviews!

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

Labels

ci CUDA/C++ Cython / Python Cython or Python issue improvement Improvement / enhancement to an existing function non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants