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

[Meshing] simplify input depth map folders #815

Merged
merged 6 commits into from
Jun 22, 2020
Merged

Conversation

dsmtE
Copy link
Contributor

@dsmtE dsmtE commented Jun 17, 2020

Description

Meshing simplification by using only one input folder for depthMaps files.

Thanks to @fabiencastan for his help.

Features list

  • Fix missing metadata for depthMap and depthMapFilter.
  • Use only one depthMaps input folder.
  • Clean histogram LOG_INFO. (use LOG_TRACE intead) [fuseCut]

dsmtE added 5 commits June 17, 2020 10:37
This metadata was only exported after depth map filter, now it is also exported after depth map estimation.
Add missing metadata after depth map filter.
Conversion of the histogram LOG_INFO to LOG_TRACE for more clarity in the information panel.
@dsmtE dsmtE marked this pull request as ready for review June 17, 2020 15:44
@fabiencastan fabiencastan added this to the 2020.1.0 milestone Jun 17, 2020
fabiencastan
fabiencastan previously approved these changes Jun 17, 2020
@@ -168,8 +167,6 @@ int aliceVision_main(int argc, char* argv[])
po::options_description optionalParams("Optional parameters");
optionalParams.add_options()
("depthMapsFolder", po::value<std::string>(&depthMapsFolder),
"Input depth maps folder.")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shouldn't the binary version be updated?

@fabiencastan fabiencastan merged commit 0449c7b into develop Jun 22, 2020
@fabiencastan fabiencastan deleted the dev/depthMapFolders branch June 22, 2020 08:11
@fabiencastan fabiencastan changed the title Dev/depth map folders [Meshing] simplify input depth map folders Aug 12, 2020
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.

3 participants