Skip to content

Commit

Permalink
PR feedback corrections
Browse files Browse the repository at this point in the history
  • Loading branch information
nv-rliu committed Mar 25, 2024
1 parent 3e20748 commit 546bdb9
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
3 changes: 1 addition & 2 deletions python/cugraph/cugraph/dask/common/part_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -115,9 +115,8 @@ def persist_dask_df_equal_parts_per_worker(
raise ValueError("return_type must be either 'dask_cudf.DataFrame' or 'dict'")

ddf_keys = dask_df.to_delayed()
workers = client.scheduler_info()["workers"].keys()
worker_to_rank = Comms.rank_to_worker(client)
# assure rank-worker mappings are in ascending order
# rank-worker mappings are in ascending order
workers = dict(sorted(worker_to_rank.items())).values()

ddf_keys_ls = _chunk_lst(ddf_keys, len(workers))
Expand Down
2 changes: 1 addition & 1 deletion python/cugraph/cugraph/dask/comms/comms.py
Original file line number Diff line number Diff line change
Expand Up @@ -269,7 +269,7 @@ def get_n_workers(sID=None, dask_worker=None):

def rank_to_worker(client):
"""
Return a mapping of dask workers to ranks.
Return a mapping of ranks to dask workers.
"""
workers = client.scheduler_info()["workers"].keys()
worker_info = __instance.worker_info(workers)
Expand Down

0 comments on commit 546bdb9

Please sign in to comment.