jraph icon indicating copy to clipboard operation
jraph copied to clipboard

Adjacency matrix to GraphsTuple

Open salfaris opened this issue 2 years ago • 4 comments

I couldn't find any function in the jraph API that allows me to convert an arbitrary adjacency matrix to a GraphsTuple object. I assume we are expected to do this manually?

Just curious if there is any potential future work on this feature? I would be happy to explore something down this road.

salfaris avatar Jun 07 '22 14:06 salfaris

Hey Salfaris,

We don't currently have a function that does this, but we would be very happy to receive pull requests :)

jg8610 avatar Jun 14 '22 09:06 jg8610

Hey @jg8610, that's great :)

I already implemented the function, but now wondering how to run the tests? It seems from the abseil documentation I have to download bazel?

salfaris avatar Jun 14 '22 12:06 salfaris

Update: nevermind I found out how to run the tests :)

salfaris avatar Jun 14 '22 13:06 salfaris

Hi @jg8610, do you mind checking this PR for me please?

salfaris avatar Jun 27 '22 11:06 salfaris

@jg8610 PR is merged (many thanks for the support!), so should we close this issue?

salfaris avatar Sep 05 '22 23:09 salfaris