Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@anmatako
Copy link
Contributor

@anmatako anmatako commented Mar 9, 2021

Extending the functionality of image_undistorter to support the following:

  1. We can now set the number of related images in the MVS workspace for patch-match through the new num_related_images option instead of always using the default of 20.
  2. We can optionally provide a text files with the image names we want included in the MVS problem through the new input_images option instead of always using all the registered images from the sparse model.
  3. We can avoid undistortion and replace it with a simple copy or linking of images when the images are already undistorted and we are not resizing them. The choice of copy or linking is controlled by the new copy_policy option.
  4. The undistorter now checks the status of each image undistortion task and only writes out the setup for the images that were successfully undistorted.

@ahojnnes ahojnnes self-requested a review March 9, 2021 22:27
@ahojnnes ahojnnes enabled auto-merge (squash) March 10, 2021 22:10
@ahojnnes ahojnnes merged commit e92320a into colmap:dev Mar 10, 2021
@anmatako anmatako deleted the public/undistortion branch March 10, 2021 23:01
lucasthahn pushed a commit to tne-ai/colmap that referenced this pull request Aug 17, 2022
* Add new functionality in image_undistorter

* Address PR comments

* Update undistortion.h

Co-authored-by: Antonios Matakos <[email protected]>
Co-authored-by: Johannes Schönberger <[email protected]>
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