Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #1340 - Use generic from_edgelist() methods #1341

Merged
merged 3 commits into from
Jan 19, 2021

Conversation

miguelusque
Copy link
Member

@miguelusque miguelusque commented Jan 17, 2021

Hi!

I have updated this notebook to make use of the new generic from_edgelist() methods. I have also updated the notebook to mention that graphs can be created from cudf, dask_cudf and Pandas dataframes.

Hope it helps!

Regards,
Miguel

closes #1340

Hi!

I have updated this notebook to make use of the new generic from_edgelist() methods. I have also updated the notebook to mention that graphs can be created from cudf, dask_cudf and Pandas dataframes.

Hope it helps!

Regards,
Miguel
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

I forgot to update the updated date.
I wrote the wrong year in the previous commit. Sorry!
@codecov-io
Copy link

codecov-io commented Jan 17, 2021

Codecov Report

Merging #1341 (d623962) into branch-0.18 (2fb0725) will increase coverage by 0.09%.
The diff coverage is 31.81%.

Impacted file tree graph

@@               Coverage Diff               @@
##           branch-0.18    #1341      +/-   ##
===============================================
+ Coverage        60.38%   60.47%   +0.09%     
===============================================
  Files               67       67              
  Lines             3029     3044      +15     
===============================================
+ Hits              1829     1841      +12     
- Misses            1200     1203       +3     
Impacted Files Coverage Δ
python/cugraph/structure/graph.py 67.18% <0.00%> (+0.38%) ⬆️
python/cugraph/utilities/utils.py 66.92% <18.18%> (-4.63%) ⬇️
python/cugraph/dask/common/input_utils.py 23.07% <28.57%> (+1.14%) ⬆️
python/cugraph/dask/common/mg_utils.py 37.50% <38.09%> (-2.50%) ⬇️
python/cugraph/structure/number_map.py 58.12% <50.00%> (+2.16%) ⬆️
...ython/cugraph/centrality/betweenness_centrality.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dcc1dfe...d623962. Read the comment docs.

@BradReesWork BradReesWork added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jan 19, 2021
@BradReesWork BradReesWork added this to the 0.18 milestone Jan 19, 2021
@BradReesWork BradReesWork merged commit 0ad18e7 into rapidsai:branch-0.18 Jan 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ENH] Update Pagerank.ipynb notebook to use generic from_edgelist() method.
3 participants