-
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
[Example] Dynamic Graph CNN on Point Cloud #789
Conversation
@BarclayII , could you please move the EdgeConv module to |
I'm actually thinking what the form of |
Temporarily closing to save CI as I'm migrating code to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work! Go ahead and merge it.
@BarclayII could you rename |
Description
This PR implements the Dynamic Graph CNN paper, as well as the
EdgeConv
module andDynamicEdgeConv
module in #748 . Namely, it providesEdgeConv
in the said paper.NearestNeighborGraph
: A module that constructs a graph from a feature matrix based on nearest neighbors.NearestNeighborGraph
andEdgeConv
yieldsDynamicEdgeConv
.)Hopefully this could be a starting point for issue #719.
This runs on ModelNet40 dataset and got a best validation accuracy of 93.5% and test accuracy of 91.8%.
Checklist
Please feel free to remove inapplicable items for your PR.
or have been fixed to be compatible with this change