-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Add option for model normals on ply export #1965
Add option for model normals on ply export #1965
Conversation
e7a3cc8
to
4f43918
Compare
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.
Thanks for adding. Sorry for the delayed review.
LGTM. Can you merge |
…on mesh export wrt normals + hoist validate pipeline up so both exporters can use it
56b6cdb
to
a5e97e9
Compare
@ethanweber updated branch with main |
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 thanks! LGTM, and I tried out the viewer changes too.
worth noting that in my case when using nerfstudio with aerial imagery the nerfacto predicted normals and the resulting mesh were significantly better looking than using open3d for estimating normals |
This PR makes the arguments related to normals for point cloud export consistent with the arguments related to normals for Poisson export. Doing this adds support for the use case where a user may want to export just a point cloud with predicted normals instead of o3d estimated normals to use in their own Poisson meshing pipeline.
Additionally, the viewer is updated to reflect the changes.