Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Export ucx dependency #173

Open
vyasr opened this issue Jan 22, 2024 · 3 comments
Open

Export ucx dependency #173

vyasr opened this issue Jan 22, 2024 · 3 comments

Comments

@vyasr
Copy link
Contributor

vyasr commented Jan 22, 2024

Once openucx/ucx#9614 is resolved we should revert #172 and #175.

@pentschev
Copy link
Member

Those changes probably won't make their way in until UCX 1.17.x, maybe 1.16.x but I'll have to check when we get the fix merged.

@vyasr
Copy link
Contributor Author

vyasr commented Feb 22, 2024

Fix was merged and backported to 1.16.x, so we can rely on it in ucxx after 1.16 is released and we bump to building against that.

@jameslamb
Copy link
Member

Noting here that if/when we make that change, it could simplify raft-dask too:

https://github.com/rapidsai/raft/blob/3720d8e91c21ec95d3dbe8e0d1a4515eb60fa7fa/python/raft-dask/CMakeLists.txt#L33-L35

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

No branches or pull requests

3 participants