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

[question] Decreasing sequential outliers #1065

Open
skinkie opened this issue Sep 15, 2020 · 8 comments
Open

[question] Decreasing sequential outliers #1065

skinkie opened this issue Sep 15, 2020 · 8 comments
Labels
do not close issue that should stay open (avoid automatically close because stale) type:question

Comments

@skinkie
Copy link

skinkie commented Sep 15, 2020

Describe the problem
This is a drone flight, as you can see there is a gap in the flight, which later appears side by side.
sequential

This is likely to happen because of a similar repeated structure, two similar radio telescopes. The results are obtained using SequentialVocubularyTree with 40 neighbours.

The basis question is: is there any way to strongly prefer sequential behavior (which I would perceive as images placed on an ordered path) over out of order movements? My understanding of the sequential neighbours was that these are the photos which are being cross checked for similar features, so having sequential neighbours = 2 does not sound right to me.

@fabiencastan
Copy link
Member

There will be a new sequential mode on the ImageMatching node in the next release, that can be combined with voctree.

@skinkie
Copy link
Author

skinkie commented Sep 16, 2020

There will be a new sequential mode on the ImageMatching node in the next release, that can be combined with voctree.

Should that be already in git?

@skinkie
Copy link
Author

skinkie commented Sep 16, 2020

Is there anything that would use timing information between frames? We are currently missing in Meshroom:

  • the ability to add a spatial heuristic
  • the ability to add a temporal heuristic

@natowi as I put in the post I have used SequentialAndVocubaryTree, and this produced this problem :(

@stale stale bot added the stale for issues that becomes stale (no solution) label Jan 14, 2021
@skinkie
Copy link
Author

skinkie commented Jan 14, 2021

Please keep open.

@stale stale bot removed the stale for issues that becomes stale (no solution) label Jan 14, 2021
@fabiencastan fabiencastan added this to the Meshroom 2021.1.0 milestone Jan 14, 2021
@alicevision alicevision deleted a comment from stale bot Jan 14, 2021
@fabiencastan fabiencastan added the do not close issue that should stay open (avoid automatically close because stale) label Jan 14, 2021
@fabiencastan
Copy link
Member

@skinkie Could you precise what you are suggesting if it is not already solved by #768?

@skinkie
Copy link
Author

skinkie commented Jan 14, 2021

@fabiencastan imagine a linear movement of a camera on a single axis to make it clear, this trajectory should be interpolatable, especially for a video. I would expect that the camera position between photo 2 and 5 should be detected as linear. I notice that meshroom does not take this into account, hence creates outliners such as 3 and 4, which does not make sense if I would consider this a trajectory. Therefore I wonder, if sequential is choose, interframe delta's should guide the trajectory, which could provide a heuristic such as a path.

image

@fabiencastan
Copy link
Member

Ah ok, so temporal continuity constraints in SfM.
Yes, that would be great to add this option for videos.

@fabiencastan fabiencastan removed this from the Meshroom 2021.1.0 milestone Feb 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not close issue that should stay open (avoid automatically close because stale) type:question
Projects
None yet
Development

No branches or pull requests

3 participants