Skip to content

[HOTFIX] Update the python API to create a PLC graph from a CSR#3027

Merged
raydouglass merged 123 commits intorapidsai:branch-22.12from
jnke2016:branch-22.12_fix_create-plc-graph-from-csr
Dec 6, 2022
Merged

[HOTFIX] Update the python API to create a PLC graph from a CSR#3027
raydouglass merged 123 commits intorapidsai:branch-22.12from
jnke2016:branch-22.12_fix_create-plc-graph-from-csr

Conversation

@jnke2016
Copy link
Contributor

@jnke2016 jnke2016 commented Dec 2, 2022

This PR enables the python layer to create a PLC graph from a CSR by leveraging the CAPI. It also update the weighted attribute if the graph is weighted.

closes #3016

@jnke2016 jnke2016 requested a review from a team as a code owner December 2, 2022 11:26
@codecov-commenter
Copy link

codecov-commenter commented Dec 2, 2022

Codecov Report

Base: 58.80% // Head: 59.41% // Increases project coverage by +0.60% 🎉

Coverage data is based on head (1eaf8f6) compared to base (1cfbfd7).
Patch coverage: 78.26% of modified lines in pull request are covered.

❗ Current head 1eaf8f6 differs from pull request most recent head 40cf600. Consider uploading reports for the commit 40cf600 to get more accurate results

Additional details and impacted files
@@               Coverage Diff                @@
##           branch-22.12    #3027      +/-   ##
================================================
+ Coverage         58.80%   59.41%   +0.60%     
================================================
  Files               131      131              
  Lines              7836     8059     +223     
================================================
+ Hits               4608     4788     +180     
- Misses             3228     3271      +43     
Impacted Files Coverage Δ
...raph/structure/graph_implementation/simpleGraph.py 79.22% <78.26%> (+0.83%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@BradReesWork BradReesWork changed the base branch from branch-22.12 to branch-23.02 December 2, 2022 20:11
@BradReesWork BradReesWork added this to the 23.02 milestone Dec 2, 2022
@rlratzel rlratzel added bug Something isn't working non-breaking Non-breaking change labels Dec 2, 2022
@rlratzel rlratzel changed the base branch from branch-23.02 to branch-22.12 December 5, 2022 23:44
@rlratzel rlratzel changed the title [WIP] Update the python API to create a PLC graph from a CSR [HOTFIX] Update the python API to create a PLC graph from a CSR Dec 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG]: create graph with from_cudf_adjlist doesn't work as intended

6 participants