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

Support Normal Estimation #43

Merged
merged 18 commits into from
Oct 26, 2023

Conversation

wweqg
Copy link
Collaborator

@wweqg wweqg commented Aug 19, 2023

Summary:

Normal Estimation Implementation

Type of change:

✨ Enhancement

Changes Made:

Implemented the normal estimation functionality to the project. The primary focus was on improving the normal estimation process and making the code more efficient, readable, and extensible. Below is a summary of the key changes made:

  1. Parallelized k-Nearest Neighbors Search using KDTree:

    • Optimized the k-nearest neighbors search using the KDTree data structure.
    • Employed the Rayon library for parallel processing, reducing computation time significantly.
  2. Covariance Matrix Computation:

    • Implemented the computation of covariance matrices for accurate normal estimation.
    • Explained the need for covariance matrices in capturing local surface structure.
  3. Eigenvalue-Eigenvector Computation:

    • Computed eigenvalues and eigenvectors from covariance matrices.
    • Clarified why the first eigenvector with the smallest eigenvalue represents the normal direction.
  4. Specialized Normal Orientation Propagation:

    • Addressed challenges related to normal orientation consistency.
    • Designed a specialized Breadth-First Search (BFS) algorithm to propagate normal orientation.

Proposed Commit Message:

Implement Normal Estimation Enhancements 

This commit introduces significant enhancements to the normal estimation process. 
Parallelization, covariance matrix computation, eigenvalue-eigenvector computation, 
and specialized normal orientation propagation have been added to improve accuracy 
and efficiency. 

@weitsang weitsang merged commit 52c7a40 into nus-vv-streams:main Oct 26, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants