-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Support for chainer? #668
Comments
Hi, thank you for the suggestion. We do love to have more backends so more users can use DGL. One question, does Chainer support DLPack protocol? It allows framework-specific tensors to expose their raw memory so DGL can perform computation on it in C. |
Sorry, I'm not sure about this. Chainer is built based on numpy and cupy. I think this could be answered by chainer developers. @kmaehashi Sorry to quoting you. I hope this won't makes you too many troubles. Could you kindly help with this issue? Thanks! |
Seems that Chainer tensors are just numpy and cupy arrays, both of which support DLPack (for cupy, see https://github.com/cupy/cupy/blob/master/docs/source/reference/interoperability.rst#dlpack). @cmpute Could you try implementing the Chainer backend? Since DGL is platform-agnostic, all you need to do to support a new backend is to create a directory For unit-testing the new backend, creating a directory |
I can have a try if time permitted. However I'm not working on graph learning recently, the progress may not be much |
No problem. I am currently working on Chainer support now. It should take a few days. |
@BarclayII for pooling layer we need more backend functions, please check https://github.com/dmlc/dgl/pull/669/files, thanks |
🚀 Feature
Support chainer as backend
Motivation
I'm a chainer user and it would be great if chainer is supported as backend. Usage of Chainer is quite similar to Pytorch, so I think it would not be that hard to port it.
Additional context
Chainer Repo
Cupy Repo
The text was updated successfully, but these errors were encountered: