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

added the PPI dataset #228

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

scratch-er
Copy link

In this pull request, we add the PPI dataset from http://snap.stanford.edu/graphsage/#datasets

The PPI dataset is loaded in the same way as Reddit. We added a new file graphsage.jl, it provides the function load_graphsage_data. Most of the code in the original reddit.jl is copied into this function, and we added labels = stack(labels) to covert the labels of the PPI dataset from Vector{JSON3.Array} to a matrix.

We added ppi.jl, which defines the PPI dataset and loads data with this function. Now reddit.jl uses this function to load data as well.

@CarloLucibello
Copy link
Member

Hi, thanks for the contribution and sorry for getting to you so late. This is a welcome addition. You just need to add the new dataset to the documentation, docs/src/datasets/graph.md

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants