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

[nodes] Masking: Handle file extensions for masks and mask inversion for ImageSegmentation #2165

Merged
merged 3 commits into from
Aug 25, 2023

Conversation

cbentejac
Copy link
Contributor

Description

This PR reflects the changes made in alicevision/AliceVision#1517 to handle other file extensions than "png" for masks and to invert mask values in the ImageSegmentation node.

Additionally, the "Camera Tracking" and "Camera Tracking and Photogrammetry" templates are updated as follows:

  • The maskInvert attribute of the ImageSegmentation node is enabled;
  • The maskInvert attribute of the FeatureExtraction node is disabled;
  • For the "Camera Tracking" pipeline, the (inverted) output of the ImageSegmentation node is connected to the PrepareDenseScene's input.

Features list

  • The MeshMasking and PrepareDenseScene nodes now have maskExtension parameters;
  • The ImageSegmentation node now has an maskInvert parameter to invert the output masks' values;
  • The "Camera Tracking" and "Camera Tracking and Photogrammetry" templates are updated.

Add the `maskExtension` parameter to specify the extension (by default,
"png") of the masks that are to be loaded.
The segmentation masks are now inverted during the `ImageSegmentation`
process, thus the `FeatureExtraction` node does not need to do it anymore.

In the CameraTracking pipeline, the `PrepareDenseScene` is now provided
with the masks from the `ImageSegmentation` node.
@cbentejac cbentejac added this to the Meshroom 2023.3.0 milestone Aug 24, 2023
@cbentejac cbentejac self-assigned this Aug 24, 2023
@mugulmd mugulmd merged commit 4890c05 into develop Aug 25, 2023
2 checks passed
@mugulmd mugulmd deleted the dev/useSegmentationMasks branch August 25, 2023 11:42
cbentejac added a commit that referenced this pull request Jan 10, 2024
Update from 1.0 to 1.2. The update to 1.1 that should have occurred with
#2165 has been forgotten.
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