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

Image Gallery: Add a menu to set the StructureFromMotion initial pair from the gallery #1936

Merged
merged 2 commits into from
Mar 18, 2023

Conversation

cbentejac
Copy link
Contributor

Description

This PR adds a "SfM: Define Initial Pair" menu in the Image Gallery, which allows the user to set the "Initial Pair A" and "Initial Pair B" parameters from the StructureFromMotion node with the currently selected image's view ID.

The menu is only enabled if there is a StructureFromMotion node in the active group.

The description of the initialPairA and initialPairB parameters in the StructureFromMotion node is also updated with the list of accepted values for these parameters (filename, filepath, or view ID).

"initialPairA" and "initialPairB" can either be the filename of the image
(without the path), the whole path of the image, or its view ID.
Add a "SfM: Define Initial Pair" sub-menu in the Image Gallery menu that is
only enabled if the currently active group contains a StructureFromMotion
node. Clicking "A" or "B" will automatically set the selected image's view
ID as a part of the initial pair.
@cbentejac cbentejac self-assigned this Mar 15, 2023
@cbentejac cbentejac changed the title [ui] Image Gallery: Add a menu to set the StructureFromMotion initial pair from the gallery Image Gallery: Add a menu to set the StructureFromMotion initial pair from the gallery Mar 15, 2023
@fabiencastan fabiencastan added this to the Meshroom 2023.1.0 milestone Mar 18, 2023
@fabiencastan fabiencastan merged commit 9d269ba into develop Mar 18, 2023
@fabiencastan fabiencastan deleted the dev/sfmSetPair branch March 18, 2023 13:46
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