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

Meshing: fix in graphcut weighting and minor sampling improvements #1520

Merged
merged 13 commits into from
Aug 28, 2023

Conversation

fabiencastan
Copy link
Member

@fabiencastan fabiencastan commented Aug 25, 2023

Features list

2 bug fixes:

  • [mvsUtils] bug fix getNbDepthValuesFromDepthMap: load the correct depth map files
  • [fuseCut] bug fix: need higher score of emptiness

1 library update:

  • Update nanoflann

Update the way we create helper points to improve performances and to improve distribution and noise level:

  • [fuseCut] helper points: inflate bbox 1.3
  • [fuseCut] helper points: add omp and make noise proportional to each axis size
  • [fuseCut] add 6 points around bbox: update from 10% to 25%
  • [fuseCut] uniform noise on helper points with 1/4 margin around the vertex
  • [fuseCut] skip some collision checks to avoid useless computation

@fabiencastan fabiencastan added this to the 3.2.0 milestone Aug 25, 2023
@fabiencastan fabiencastan marked this pull request as ready for review August 25, 2023 13:15
for(int camid = 0; camid < cams.size(); camid++)
{
int rc = cams[camid];
{
const Point3d p(_mp.CArr[rc].x, _mp.CArr[rc].y, _mp.CArr[rc].z);
std::size_t vi;
double sq_dist;
// std::size_t vi;
Copy link
Contributor

@mugulmd mugulmd Aug 28, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there any reason to keep the code that's been commented out ?

@mugulmd mugulmd merged commit 3d068fe into develop Aug 28, 2023
3 of 4 checks passed
@mugulmd mugulmd deleted the fix/meshingGraphCut branch August 28, 2023 11:29
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