gyroflow icon indicating copy to clipboard operation
gyroflow copied to clipboard

Dynamic zooming zooms in too much for lens correction strengths other than 0 and 100%

Open avsaase opened this issue 3 years ago • 6 comments
trafficstars

Is there an existing issue for this?

  • [X] I have searched the existing issues

Have you tried the latest build?

  • [X] I have tried the latest build

Gyroflow version

72b8eb6a64d80292d1006277c4d4212865fca072

What operating system are you using?

Windows 10

What GPU are you using?

No response

What happened?

When selecting lens correction strengths other than 0 and 100% dynamic and fixed zooming zooms in too much. Take for example this clip without motion from the Firefly X Lite II on 2.7K 4:3.

0% lens correction: image

100% lens correction: image

50% lens correction: image

As you can see in the 50% lens correction example there is FOV "left on the table". This happens with all settings between 0 and 100% but seems to be maximized at or around 50%.

I increased the FOV to 1.5 to make it more obvious but it's not a bug in the Safe area guide because it also happens with FOV=1. It's also not related to the output size setting because I tried setting that to the input resolution but that didn't make a difference. I tried a couple of different lens profiles for this camera and setting, all with the same result. I recall seeing this with the Mobius Maxi 4K as well.

Here's the clip and .gyroflow file: https://mega.nz/folder/00NwGYwL#TAKzsDrmqTZ-FLzTkrvhIg

Relevant log output

No response

avsaase avatar Jun 22 '22 21:06 avsaase

I know, this is because I don't know how to calculate camera params when adding lens distortion back, so currently it's something approximated, see: https://github.com/gyroflow/gyroflow/blob/088b02cf10b4a900f9e54aa14222f8183442bfec/src/core/gpu/opencl_undistort.cl#L275

This will likely stay this way until someone smarter than me figures it out

AdrianEddy avatar Jun 22 '22 21:06 AdrianEddy

/bounty $150

AdrianEddy avatar Sep 22 '23 01:09 AdrianEddy

💎 $150 bounty created by gyroflow 🙋 If you start working on this, comment /attempt #384 to notify everyone 👉 To claim this bounty, submit a pull request that includes the text /claim #384 somewhere in its body 📝 Before proceeding, please make sure you can receive payouts in your country 💵 Payment arrives in your account 2-5 days after the bounty is rewarded 💯 You keep 100% of the bounty award 🙏 Thank you for contributing to gyroflow/gyroflow!

Attempt Started (GMT+0) Solution
🟢 @aazam-gh Oct 1, 2023, 8:50:00 AM WIP
🟢 @Rutik7066 Dec 25, 2023, 8:21:50 AM WIP

algora-pbc[bot] avatar Sep 22 '23 01:09 algora-pbc[bot]

/attempt #384

Options

aazam-gh avatar Oct 01 '23 08:10 aazam-gh

/attempt #384 got it.

Options

rutikthakre avatar Dec 25 '23 08:12 rutikthakre

[!NOTE] The user @aazam-gh is already attempting to complete issue #384 and claim the bounty. We recommend checking in on @aazam-gh's progress, and potentially collaborating, before starting a new solution.

algora-pbc[bot] avatar Dec 25 '23 08:12 algora-pbc[bot]