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.

Sign up for GitHub

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

Jump to bottom

Pass intrinsic calibration as prior information #1033

Open
tompollok opened this issue Oct 29, 2020 · 6 comments · May be fixed by #1166
Open

Pass intrinsic calibration as prior information #1033

tompollok opened this issue Oct 29, 2020 · 6 comments · May be fixed by #1166

Comments

@tompollok
Copy link
Contributor

Is there a way to pass colmap an intrinsic calibration for specific images and even maybe prevent optimization during BA?
@tsattler
Copy link
Contributor

Yes, you can specify intrinsics both from the command line and in the gui. You can also fix the intrinsic parameters during the reconstruction. However, this will fix all intrinsics.

@DaniilSNikulin
Copy link
Contributor

DaniilSNikulin commented Oct 30, 2020 •
edited

Colmap already have parameter --ImageReader.single_camera_per_folder.
In private fork I have parameter --camera_params_per_folder_path which specify file with mapping 'folder name -> intrinsics'.
I could provide PR with this parameter if COLMAP owners (@ahojnnes) will be agree.

@ahojnnes
Copy link
Contributor

ahojnnes commented Oct 31, 2020 via email

@tompollok
Copy link
Contributor Author

That sounds great!

@tompollok
Copy link
Contributor Author

Btw is anywhere an evaluation of how sfm / BA based intrinsic calibration compared to classical chessboard approaches? I guess it depends on the number of overlapping images and correspondences found. Did anybody look into this in the past?

@tompollok
Copy link
Contributor Author

@DaniilSNikulin any plans to make a PR anytime soon?

@DaniilSNikulin DaniilSNikulin linked a pull request Mar 9, 2021 that will close this issue
Open
@Li-colonel Li-colonel mentioned this issue Apr 6, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Issue 1033 prior intrinsic for multiple cameras DaniilSNikulin/colmap
4 participants
@ahojnnes @tompollok @tsattler @DaniilSNikulin