Skip to content
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

Global Pooling via EquilibriumAggregation #4447

Closed
rusty1s opened this issue Apr 10, 2022 · 4 comments
Closed

Global Pooling via EquilibriumAggregation #4447

rusty1s opened this issue Apr 10, 2022 · 4 comments

Comments

@rusty1s
Copy link
Member

rusty1s commented Apr 10, 2022

🚀 The feature, motivation and pitch

Support the recently proposed "Equilibrium Aggregation" scheme which defines an implicitly given global readout function. The blog post goes over it in detail.

  • Paper: The code is given in the appendix.

Alternatives

No response

Additional context

No response

@Padarn
Copy link
Contributor

Padarn commented Apr 17, 2022

This paper is pretty cool. I'm happy to start on this.

I wonder if its worth also trying to implement it as a possible aggregation function in message passing, rather than just as a global readout. That way the global readout version could be implemented just by passing a different adjacency matrix to the layer?

@rusty1s
Copy link
Member Author

rusty1s commented Apr 17, 2022

I think providing it as a global readout layer is sufficient for now. I don‘t think we need to add it as a general option for aggregation in any message passing layer for now.

@Padarn
Copy link
Contributor

Padarn commented Apr 18, 2022

Fair enough, worth doing separately

@Padarn
Copy link
Contributor

Padarn commented Apr 24, 2022

Have opened a PR, and assigned this to myself - still a number of open questions to discuss and a lot of cleaning to do.

@rusty1s rusty1s added nn and removed pooling labels Aug 31, 2022
@rusty1s rusty1s closed this as completed Sep 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants