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

[FEA] communicator abstraction #814

Closed
afender opened this issue Apr 6, 2020 · 2 comments
Closed

[FEA] communicator abstraction #814

afender opened this issue Apr 6, 2020 · 2 comments
Milestone

Comments

@afender
Copy link
Member

afender commented Apr 6, 2020

We should have a communicator abstraction regardless of what backend is used. This allows our code to be independent of the actual communication framework.

The communicator would have features spanning from comm.get_rank() to comm.allReduce(...)

We should start from RAFT see #523 and rapidsai/raft#7
In the long run, we should allow multiple communicators and host/device targets.

@afender afender added cuda ? - Needs Triage Need team to review and classify labels Apr 6, 2020
@BradReesWork BradReesWork removed the ? - Needs Triage Need team to review and classify label Apr 13, 2020
@BradReesWork BradReesWork added this to the 0.15 milestone Apr 13, 2020
@afender afender mentioned this issue May 26, 2020
@afender
Copy link
Member Author

afender commented May 28, 2020

While #840 added the C++ temporary communicator in 0.14. We should repurpose this issue to integrate RAFT comms

@afender afender changed the title [FEA] C++ communicator abstraction [FEA] communicator abstraction May 28, 2020
@afender
Copy link
Member Author

afender commented Jun 19, 2020

done #947 #944 #896 .
Temporary comm was canceled, now using RAFT

@afender afender closed this as completed Jun 19, 2020
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

2 participants