-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
Sparse version of nn.dense.dense_mincut_pool
#9118
Comments
If you see any major flaw in this idea, please let me know. I'd be glad to contribute to this idea. |
Similar issue: #7364 |
We initially designed it this way because |
Hi, I'm interested in making a pull request. Meanwhile I cannot push my branch to this project. Maybe I need to be added to this project first? |
If you want to contribute to PyG, you need to fork this repo, push the changes to your local fork, and then submit a PR to PyG. |
Thanks for the tips. I made a draft PR #9243 but have some questions. It would be great if someone could take a look at it. |
🚀 The feature, motivation and pitch
The nn.dense.dense_mincut_pool operator requires the input adjacency matrix to be dense. This requirement does not scale well as the matrix size grows. We may use operations on sparse matrix to address this issue.
Alternatives
Additional context
No response
The text was updated successfully, but these errors were encountered: