Skip to content

similarity notebook to compare link prediction algos#3868

Merged
rapids-bot[bot] merged 5 commits intorapidsai:branch-23.10from
acostadon:similarity_nb
Sep 26, 2023
Merged

similarity notebook to compare link prediction algos#3868
rapids-bot[bot] merged 5 commits intorapidsai:branch-23.10from
acostadon:similarity_nb

Conversation

@acostadon
Copy link
Contributor

@acostadon acostadon commented Sep 21, 2023

New notebook to compare link prediction
Dependent on dining_prefs being added to datasets api in PR #3866

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@acostadon acostadon self-assigned this Sep 21, 2023
@acostadon acostadon added feature request New feature or request non-breaking Non-breaking change labels Sep 21, 2023
@acostadon acostadon added this to the 23.10 milestone Sep 21, 2023
@acostadon acostadon added DO NOT MERGE Hold off on merging; see PR for details and removed DO NOT MERGE Hold off on merging; see PR for details labels Sep 21, 2023
@acostadon acostadon marked this pull request as ready for review September 21, 2023 17:41
@acostadon acostadon requested a review from a team as a code owner September 21, 2023 17:41
@BradReesWork
Copy link
Member

Cell 4 - cugraph is imported again
Cell5 - cugraph is imported again

@BradReesWork
Copy link
Member

datasets has a "get_graph" functions that we should use rather than getting the edge list and building a graph

},
"nbformat": 4,
"nbformat_minor": 2
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

better documentation

@BradReesWork
Copy link
Member

/merge

@rapids-bot rapids-bot bot merged commit c11eff2 into rapidsai:branch-23.10 Sep 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature request New feature or request non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants