Closed aramakus closed 1 year ago
Hi @aramakus , thanks for your help! We are more than happy to add this feature to dgl. You can follow this instruction to start contributing to dgl. Let us know if you have any challenges when you try to contribute, we are working on improving the experience in helping community contributors.
Hi @frozenbugs , Thanks for a quick reply. I have followed the instruction, here is the PR https://github.com/dmlc/dgl/pull/4689 please let me know if there are any issues with code. I will happily provide feedback once I understand the process a little better.
This issue has been automatically marked as stale due to lack of activity. It will be closed if no further activity occurs. Thank you
This issue has been automatically marked as stale due to lack of activity. It will be closed if no further activity occurs. Thank you
I am closing this issue due to lack of activity. Feel free to follow up and reopen the issue if you have more questions with regard to our response.
Hi Minjie, certainly. After spending about 80% of spare time that I allocated to this PR trying to setup unit test environment without success I do intend to proceed. The code for EdgeWeightNorm is complete, only local unit tests remain. If anyone wants to pick up from here, they are welcome to it.
🚀 Feature
Support for weighted graphs with Tensorflow backend
Motivation
DGL users, who use Tensorflow backend are currently need to switch to PyTorch if they need to access weighted graph functionality. A simple code change that I attach here extends this functionality to Tensorflow users as well.
Pitch
I have written the code for this feature, but do not know how to join the developers community and contribute to the library. Here is the code, it largely reproduces that of PyTorch backend version:
Additional context
Like with PyTorch implementation, this file can be used for the code
dgl/python/dgl/nn/tensorflow/conv/graphconv.py