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

Add visualisation mode for landmarks observations #2177

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

Conversation

almarouk
Copy link
Member

@almarouk almarouk commented Sep 4, 2023

Linked to alicevision/QtAliceVision#42

  • add an option to visualize the landmarks observations
  • observed landmarks for current camera + links to cameras observing the same landmarks
  • linked to the 2D viewer to zoom on specific landmarks

landmarks

@natowi
Copy link
Member

natowi commented Sep 4, 2023

It would be great if an edit mode could be added on top of this feature in a following contribution :)

@almarouk almarouk force-pushed the dev/vizObservations branch 2 times, most recently from 15addf6 to 55c1f9f Compare September 18, 2023 09:13
@almarouk almarouk force-pushed the dev/vizObservations branch 2 times, most recently from 291c369 to 268b6bd Compare September 22, 2023 07:51
@simogasp
Copy link
Member

That's very interesting!
Another interesting feature would be colour-coding the 3D points based on their visibility (i.e., the number of observations). That gives a hint of how "reliable" the 3D point is.

@almarouk almarouk force-pushed the dev/vizObservations branch 2 times, most recently from dd7b449 to e5e7213 Compare September 26, 2023 15:15
@almarouk almarouk force-pushed the dev/vizObservations branch 2 times, most recently from a1e1621 to 19c6f54 Compare October 12, 2023 08:45
@almarouk almarouk force-pushed the dev/vizObservations branch 2 times, most recently from 23540dc to 94e8ed3 Compare March 26, 2024 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants