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.

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

Focal length has 2 values #167

Open
ehedlin opened this issue May 1, 2020 · 1 comment
Open

Focal length has 2 values #167

ehedlin opened this issue May 1, 2020 · 1 comment

Comments

@ehedlin
Copy link

ehedlin commented May 1, 2020

Hey, I have a question about cameras.h5. Each camera seems to have 2 focal lengths (shape 2x1). All other entries seem to make sense to me (translation is a 3x1, rotation matrix is a 3x3x1, etc). Which is the right value or is there something I am missing?

@una-dinosauria
Copy link
Owner

Great catch! Apparently our documentation is wrong 😅

They are both right. Often the x and y components of the focal length can differ due to a number of reasons; eg from http://ksimek.github.io/2013/08/13/intrinsic/

  • Flaws in the digital camera sensor.
  • The image has been non-uniformly scaled in post-processing.
  • The camera's lens introduces unintentional distortion.
  • The camera uses an anamorphic format, where the lens compresses a widescreen scene into a standard-sized sensor.
  • Errors in camera calibration.

I'll be very thankful if you submit a PR correcting the documentation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants