B1ueber2y ยท GitHub

Summary of Changes

Hello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request enhances the triangulation process in COLMAP by enabling the loading of keypoints for all images, regardless of whether they have valid two-view geometry. This is achieved through the introduction of a load_all_images option, which, when enabled, ensures that keypoints from all images are available for triangulation, potentially benefiting structure-less registration and localization of new images.

Highlights

  • Keypoint Loading: This PR introduces a new option to load keypoints for all images into the database cache, even those without valid two-view geometry.
  • Triangulation Enhancement: The primary goal is to improve triangulation with known poses by utilizing keypoints from images that might otherwise be excluded due to lack of two-view geometry.
  • New load_all_images Option: A new boolean option, load_all_images, is added to control whether all images are loaded, regardless of correspondences. This is exposed in the C++ and Python APIs.
Changelog
  • src/colmap/controllers/incremental_pipeline.cc
    • Added load_all_images option to DatabaseCacheOptions.
  • src/colmap/controllers/incremental_pipeline.h
    • Introduced load_all_images option to IncrementalPipelineOptions to control loading of all images from the database.
  • src/colmap/exe/sfm.cc
    • Set load_all_images to true for point triangulator.
  • src/colmap/scene/database_cache.cc
    • Modified database cache loading logic to conditionally load all images based on the load_all_images option.
    • Updated logging to include the number of connected and loaded images.
  • src/colmap/scene/database_cache.h
    • Added load_all_images option to the DatabaseCache::Options struct.
  • src/pycolmap/scene/database_cache.cc
    • Exposed the load_all_images option to the Python API.
  • src/pycolmap/sfm/incremental_mapper.cc
    • Exposed the load_all_images option to the Python API for incremental pipeline.
Activity
  • The pull request introduces a new feature to load keypoints for all images, regardless of whether they have valid two-view geometry.
  • The load_all_images option is added to both C++ and Python APIs.
  • The code changes modify the database cache loading logic to conditionally load all images based on the new option.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with ๐Ÿ‘ and ๐Ÿ‘Ž on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. โ†ฉ

Read the original on github.com โ†—