cugraph icon indicating copy to clipboard operation
cugraph copied to clipboard

[ENH] Add ability to re-use constructed C++ `graph_t` instances from Python

Open rlratzel opened this issue 3 years ago • 3 comments

Current cuGraph Python APIs that require a call to an underlying C++ analytic will create a C++ graph_t instance prior to calling the C++ function, convert results to a Python/cuDF type, then delete the graph_t. This happens for each Python API call even if the intermediate graph_t is identical to the prior call. Constructing a graph_t can be expensive, and the time to construct it inflates algo run times.

note that graph_t instances are different than the pure-python Graph class instances, which are currently explicitly created and reused.

This issue tracks the work involved in allowing the Python layer to re-use graph_t instances if possible across algo calls. An additional API will also be provided to allow a user to construct the graph_t upfront in order to better track performance improvements/regressions of the underlying C++ code from python, since the time to construct a graph_t will then be excluded.

cc @seunghwak @ChuckHastings @BradReesWork

rlratzel avatar Jan 13 '22 18:01 rlratzel

This issue has been labeled inactive-30d due to no recent activity in the past 30 days. Please close this issue if no further response or action is needed. Otherwise, please respond with a comment indicating any updates or changes to the original issue and/or confirm this issue still needs to be addressed. This issue will be labeled inactive-90d if there is no activity in the next 60 days.

github-actions[bot] avatar Feb 12 '22 19:02 github-actions[bot]

This issue has been labeled inactive-30d due to no recent activity in the past 30 days. Please close this issue if no further response or action is needed. Otherwise, please respond with a comment indicating any updates or changes to the original issue and/or confirm this issue still needs to be addressed. This issue will be labeled inactive-90d if there is no activity in the next 60 days.

github-actions[bot] avatar Apr 10 '22 18:04 github-actions[bot]

This issue has been labeled inactive-90d due to no recent activity in the past 90 days. Please close this issue if no further response or action is needed. Otherwise, please respond with a comment indicating any updates or changes to the original issue and/or confirm this issue still needs to be addressed.

github-actions[bot] avatar Aug 30 '22 14:08 github-actions[bot]