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

renew curvature algorithms for mesh & pcl #1560

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Luo-Yihao
Copy link

@Luo-Yihao Luo-Yihao commented Jun 14, 2023

[WIP] Start off by expressing my gratitude for the amazing pytorch3d library. However, while working with pytorch3d.ops.estimate_pointcloud_local_coord_frames function, I have noticed an issue with the description of the principal curvatures. More technically, the eigenvector corresponding to the minimal eigenvalue of covariance is exactly the normal vector, but neither the other two eigenvalues are principal curvatures. The current implementation does not accurately compute the curvature, which is a crucial metric in the field of 3D deep learning. Therefore, I have taken the liberty of developing codes for curvature algorithms based on pytorch3d for both meshes and point clouds. I believe that this algorithm would be a valuable addition to the library and would help fill the current gap in the available curvature computation methods.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jun 14, 2023
@Luo-Yihao Luo-Yihao marked this pull request as ready for review June 14, 2023 23:09
@Luo-Yihao
Copy link
Author

Please check my private repository Luo-Yihao/Torch3d_Curvature, which contains a detailed 'tutorial.ipynb' for how to use the codebase.

@Luo-Yihao Luo-Yihao closed this Jul 5, 2023
@Luo-Yihao Luo-Yihao reopened this Jul 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants