[REVIEW] BUG Move subcomms init outside of individual algorithm functions#1196

Merged

BradReesWork merged 13 commits intorapidsai:branch-0.16from rlratzel:branch-0.16-movesubcomminit

Oct 9, 2020

+184-85

Commits on Oct 6, 2020

WIP: initial commit, incomplete
rlratzel
committed

Commits on Oct 7, 2020

Initial version that builds using a new cythin C++ util to initi subcomms. Still have an issue with error stating comms not initialized when trying to init subcomms.
rlratzel
committed
Merge remote-tracking branch 'upstream/branch-0.16' into branch-0.16-movesubcomminit
rlratzel
committed
WIP update to move subcomm init to user-facing comms init call.
rlratzel
committed
move subcomm init to comms init
Iroy30
committed
updates
Iroy30
committed
Removing FIXMEs and old code now that subcomms init is moved to comms init.
rlratzel
committed
Merge remote-tracking branch 'origin/branch-0.16-movesubcomminit' into branch-0.16-movesubcomminit
rlratzel
committed
Added PR 1196 to CHANGELOG.md, removed additional obsolete FIXME
rlratzel
committed
Addressed FIXME by removing redundant prows and pcols args to populate_graph_container() call and using values obtained directly from handle instead.
rlratzel
committed
C++ style check updates
rlratzel
committed
Updated docs and comments based on review feedback. Minor consolidation of get_n_workers() call, but still need FIXME for that addressed.
rlratzel
committed
flake8 updates
rlratzel
committed