ahojnnes · GitHub

@ahojnnes

Copy link Copy Markdown

Contributor

Currently, if we manage to initialize from one pair but get stuck after a few images and none of the other pairs satisfy the initialization thresholds, then we never try to initialize from a relaxed set of thresholds anymore for the remaining images, even though there is a high chance to succeed. In addition, we unnecessarily tried the same pairs redundantly for the same initialization constraints. This should result in more complete reconstruction results and faster initialization.

@ahojnnes

sarlinpe

Copy link Copy Markdown

Member

Choose a reason for hiding this comment

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

Nice improvements, thank you! LGTM, but let's wait that @B1ueber2y reviews too.

sarlinpe

B1ueber2y

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.

Thanks. LGTM

@ahojnnes

@ahojnnes

@ahojnnes

ahojnnes deleted the user/jsch/init-mapper-improvements branch

February 17, 2025 11:08

Merged

Read the original on github.com ↗