-
Notifications
You must be signed in to change notification settings - Fork 43
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
Meaningfulness of center dot #144
Comments
capricorn8
added a commit
to capricorn8/Focus-Points
that referenced
this issue
Jan 1, 2025
Issue musselwhizzle#144 and related fix was nonsense. For Olympus/OM cameras, the only useful information related to focus point is AFPointSelected. Drawing a box around this point has no added meaning in terms of focusing, but it helps to recognize / find the point more easily on the image. Reverting implementation to POINTTYPE_AF_SELECTED_INFOCUS.
Opening this issue was nonsense. For Olympus/OM cameras, the only useful information related to focus point is AFPointSelected. Drawing a box around this point has no added meaning in terms of focusing, but it helps to recognize / find the point more easily on the image. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
For some cameras (Olympus OM-D, probably other mirrorless cameras as well) the center dot doesn't make sense; it's rather confusing because it suggests that this spot in the image is particularly important in terms of focusing whereas it's not.
In case of Olympus OM-D cameras, the "AF Areas" information is just one of the predefined AF fields. The exact focus point can be anywhere within the rectangle; it is determined by the camera's AF algorithm and its exact position is not available in EXIF data.
Not sure if this is something that can be handled universally by the plugin, in a transparent manner. Maybe it's too complex. Another option would be to make it an option in the plugin whether to use center dots or not. Just as an idea.
(as a quick fix I have commented out this portion in the code I'm using)
The text was updated successfully, but these errors were encountered: