search

musselwhizzle / Focus-Points

Plugin for Lightroom to show which focus point was active in the camera when a photo was taken
Apache License 2.0
345 stars 42 forks source link

Improved support for Sony RX10M4 #159

Closed jodaco67 closed 3 years ago

jodaco67 commented 3 years ago

The results for my RX10M4 seemed inaccurate and the active focus point never aligned to one of the pdaf points. There is a windows program called AFV (https://github.com/SK-Hardwired/s_afv) that shows different results. Comparing the algorithms I think the approach in that program is better since it doesn't require arbitrary parameters like pdafScale but instead exclusively uses image geometry. I applied this to my own code and it works very nicely and now the in-focus points almost always aligns with one of the pdaf points. I still need to implement face and animal eye notation but I can do that later.

Since I only have a RX10M4 and no other Sony cameras I can't be sure this works in all cases so I created a SonyRX10M4Delegates and modified the PointsRendererFactory accordingly. I don't see any other model specific delegates so I'm not certain this is the correct approach. I'm happy to share my code with the community if these changes are acceptable.

musselwhizzle commented 3 years ago

Hi, Would you mind creating a pull request with your changes!?

jodaco67 commented 3 years ago

Please forgive my ignorance. I created a branch in GitHub Desktop called '159' (my issue number) but when I try to create a pull request it says I must publish. When i try to do that it says I do not have write permissions. I tried the commands in Contributing.md but I'm getting various errors there too.

musselwhizzle commented 3 years ago

If you can zip up the contents and attach them here, I can add the code.

On Sat, Jun 5, 2021, 8:33 PM jodaco67 @.***> wrote:

Please forgive my ignorance. I created a branch in GitHub Desktop called '159' (my issue number) but when I try to create a pull request it says I must publish. When i try to do that it says I do not have write permissions. I tried the commands in Contributing.md but I'm getting various errors there too.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/musselwhizzle/Focus-Points/issues/159#issuecomment-855316286, or unsubscribe https://github.com/notifications/unsubscribe-auth/AARTUQRNKQNCHGISKYMTZF3TRK67LANCNFSM46E5JKJA .

jodaco67 commented 3 years ago

here you go changedFiles.zip

musselwhizzle commented 3 years ago

https://github.com/musselwhizzle/Focus-Points/pull/161

Merged