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.

Sign up for GitHub

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

Jump to bottom

Viewer color source #1195

Closed
wants to merge 6 commits into from
Closed

Viewer color source #1195

wants to merge 6 commits into from

Conversation

JuSiGabriel
Copy link
Contributor

Adding visualisation of confidence map on point cloud in Viewer. Different confidence maps are avaible via F1 to F6 key.

  • F1 : no color ;
  • F2 : image color ;
  • F3 : original confidence-map (ZNCC) ;
  • F4 : confidence estimated from depth-map. The estimation is the mean depth difference between the main pixel and the other pixels of a 3X3 window. The mean is computed only from the 3 lower differences. This choice minimizes edge effect.
  • F5 : combination of original confidence (w1) and confidence estimated from depth-map (w2) (0.3w1 + 0.7w2)
  • F6 : confidence from normal variance in a 3x3 window

cdcseacave and others added 6 commits November 8, 2024 13:34
@cdcseacave
mesh: make TRasterMesh multithread-safe
384da63 
(cherry picked from commit bbfae87948dc0fa128b1a42c040c5d5e58e4ec0c)
@cdcseacave
mesh: add multi-threaded mesh projection test
3424afd 
(cherry picked from commit 6eb560b4b692624c7bd77f8151b8d1f6276fa881)
@cdcseacave
Merge remote-tracking branch 'origin/develop' into develop_justin
f320a01 
# Conflicts:
#	apps/Tests/Tests.cpp
#	libs/MVS/Mesh.cpp
@JuSiGabriel
Merge remote-tracking branch 'base/develop' into develop_justin
f69109c
@JuSiGabriel
Add visualization of various confidence maps in color of the point cl…
ccd4a30 
…oud in the Viewer.
@JuSiGabriel
Refactor and clean
edb3e89
@cdcseacave
Copy link
Owner

merged in develop, thank you

@cdcseacave cdcseacave closed this Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@JuSiGabriel @cdcseacave