update FIXME with more detail

rapidsai · May 11, 2022 · 445d019 · 445d019
1 parent 2b47100
commit 445d019
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/python/cugraph/cugraph/dask/link_analysis/hits.py b/python/cugraph/cugraph/dask/link_analysis/hits.py
@@ -155,6 +155,9 @@ def hits(input_graph, tol=1.0e-5, max_iter=100,  nstart=None, normalized=True):
     client = default_client()
 
     # FIXME: 'legacy_renum_only' will not trigger the C++ renumbering
+    # In the future, once all the algos follow the C/Pylibcugraph path,
+    # compute_renumber_edge_list will only be used for multicolumn and
+    # string vertices since the renumbering will be done in pylibcugraph
     input_graph.compute_renumber_edge_list(
         transposed=False, legacy_renum_only=True)
     ddf = input_graph.edgelist.edgelist_df

diff --git a/python/cugraph/cugraph/dask/sampling/neighborhood_sampling.py b/python/cugraph/cugraph/dask/sampling/neighborhood_sampling.py
@@ -129,6 +129,9 @@ def EXPERIMENTAL__uniform_neighborhood(input_graph,
     # Initialize dask client
     client = default_client()
     # FIXME: 'legacy_renum_only' will not trigger the C++ renumbering
+    # In the future, once all the algos follow the C/Pylibcugraph path,
+    # compute_renumber_edge_list will only be used for multicolumn and
+    # string vertices since the renumbering will be done in pylibcugraph
     input_graph.compute_renumber_edge_list(
         transposed=False, legacy_renum_only=True)