Tristan Bilot
Tristan Bilot
I'll take OpenAIGPT!
The idea seems cool, but this would require a lot of work to implement all message passing functions in C++, scattering etc. Are there any sparse operations implemented yet in...
Thanks @DaveSprague for reporting this issue, it was indeed an important error!
Hi @dgreda :) thanks for your interest! I didn't think of this new feature yet but you're welcome to make a PR if you wish to contribute. Do you have...
Anyone working on this?
As mentioned [here](https://github.com/ml-explore/mlx/pull/380#issuecomment-1879723940) by @gboduljak, an important question we may ask before diving into the implementation of various GNN implementations is: Should we create a separate library specifically made for...
Yes, implementing all `scatter_*` operations will be one of the main things to achieve. I plan to be working on that too.
I think that the `mlx-examples` option is a no go, as this PR goes beyond just releasing a new model. The MPNN is one of the fundamental basic blocks that...
It would probably be much easier for maintenance to have all GNN-related code within the separate `mlx-graphs` lib (MPNN, GraphNet, etc), and the basic operations (scattering, sparse operations) remain in...
@awni, I think this is a wonderful idea! And it is IMO the most logical and straightforward way to do. I personally can allocate a lot of time on this...