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

Adds an example for Hierarchical Sampling #7244

Merged

Conversation

andreazanetti
Copy link
Contributor

It compares one epoch of training with and without Hierarchical Sampling.
With pyg-lib>0.1.0 we return the sampled number of nodes/edges in neighbor_sampler.py
Leveraging this, the training_benchmark.py refers to BasicGNN base class, in which the forward pass does the trimming if required (using the --trim flag with training_benchmark.py).
Therefore, this is an example that mimics what is being done in the training_benchmark.py, to make evident for the user what this trimming/Hierarchical Sampling is about, how to test it, and have an idea of the advantage.

@rusty1s rusty1s changed the title It adds a minimal example of Hierarichial Sampling on one Epoch. Adds a minimal example of Hierarichial Sampling Apr 28, 2023
@andreazanetti
Copy link
Contributor Author

@rusty1s do you have comments on this simple example? Thank you!

@rusty1s rusty1s changed the title Adds a minimal example of Hierarichial Sampling Adds a minimal example of Hierarchical Sampling May 10, 2023
@rusty1s rusty1s changed the title Adds a minimal example of Hierarchical Sampling Adds an example for Hierarchical Sampling May 10, 2023
@rusty1s rusty1s marked this pull request as ready for review May 10, 2023 16:10
@rusty1s rusty1s requested a review from wsad1 as a code owner May 10, 2023 16:10
@rusty1s rusty1s enabled auto-merge (squash) May 10, 2023 16:11
@rusty1s rusty1s merged commit 82f31cf into pyg-team:master May 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants