Skip to content

Make global positioning scale and weighting policies configurable - #4763

Open
Zador-Pataki wants to merge 9 commits into
colmap:mainfrom
Zador-Pataki:zador/global-positioning-controls
Open

Zador-Pataki wants to merge 9 commits into
colmap:mainfrom
Zador-Pataki:zador/global-positioning-controls

Conversation

@Zador-Pataki

@Zador-Pataki Zador-Pataki commented Sep 20, 2026

Copy link
Copy Markdown
Contributor

Motivation

Global positioning hard-codes scale-gauge fixing and uncalibrated-camera weighting, and couples scale initialization to point randomization. These choices restrict downstream research, for example when extending global positioning with additional residuals.

Changes

  • Remove the dependency of scale initialization on point randomization and optimization, and rename and invert generate_scales to initialize_scales_from_geometry.
  • Replace the hard-coded uncalibrated-observation loss multiplier with uncalibrated_observation_weight (default 0.5). 1.0 gives calibrated and uncalibrated observations equal weight.
  • Allow callers to disable the previously unconditional fixing of the first observation scale with fix_first_scale (default true).

Comment thread src/colmap/estimators/global_positioning.cc Outdated
Comment thread src/colmap/estimators/global_positioning.cc Outdated
Comment thread src/colmap/estimators/global_positioning.h Outdated
Comment thread src/colmap/estimators/global_positioning_test.cc Outdated
Comment thread src/colmap/estimators/global_positioning_test.cc
Comment thread src/colmap/estimators/global_positioning.cc
@Zador-Pataki

Copy link
Copy Markdown
Contributor Author

@B1ueber2y thanks! I addressed everything and clarified the remaining open question.

@B1ueber2y B1ueber2y left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. Thanks!

Comment thread src/colmap/estimators/global_positioning.h Outdated
Comment thread src/colmap/estimators/global_positioning.cc

This branch has not been deployed

No deployments
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