Alexander Brock Alexander-Brock
  • I studied computer science at Heidelberg University. Now I'm working on camera calibration for RabbitAI GmbH.

  • Joined on 2024-06-17
Alexander Brock commented on pull request blender/blender#123737 2024-06-28 10:46:51 +02:00
Fix and test direction_to_fisheye_lens_polynomial

I accidentally used the GNU extensions for math constants. I think I fixed it, could someone trigger the build?

Alexander Brock pushed to test-flepo at Alexander-Brock/blender 2024-06-28 01:00:23 +02:00
9174bf20c8 Remove early return and use safe_normalize in fisheye lens polynomial
Alexander Brock commented on pull request blender/blender#123737 2024-06-28 00:55:01 +02:00
Fix and test direction_to_fisheye_lens_polynomial

I removed the early return and use safe_normalize, could you trigger the build so I can look more into why the tests fail on Windows?

Alexander Brock commented on pull request blender/blender#123737 2024-06-28 00:11:23 +02:00
Fix and test direction_to_fisheye_lens_polynomial

I agree and changed it, but it lead to NaN being returned in the center, so I added early return near the center:

  if (dir.y < 1e-8f && dir.y > -1e-8f && dir.z < 1e-8f && dir.z > -1e-8f)…
Alexander Brock pushed to test-flepo at Alexander-Brock/blender 2024-06-28 00:11:09 +02:00
78883205ab Reduce trigonometry and return early near the center in fisheye lens polynomial
Alexander Brock pushed to test-flepo at Alexander-Brock/blender 2024-06-28 00:03:27 +02:00
00daafa474 Combine two short lines into one
Alexander Brock pushed to test-flepo at Alexander-Brock/blender 2024-06-27 23:57:11 +02:00
4fffe43742 Pull differential coefficients out of loop and compute F_r and dF_r separately in fisheye lens polynomial
Alexander Brock pushed to test-flepo at Alexander-Brock/blender 2024-06-27 19:17:58 +02:00
73f03cd1d5 Formating changes only
Alexander Brock pushed to test-flepo at Alexander-Brock/blender 2024-06-27 19:06:17 +02:00
be6f46ec25 Explicitly add center in test for fisheye lens polynomial
Alexander Brock pushed to test-flepo at Alexander-Brock/blender 2024-06-27 19:05:03 +02:00
548f4518e5 Add diagonal test case to fisheye lens polynomial
Alexander Brock commented on pull request blender/blender#123737 2024-06-27 17:47:59 +02:00
Fix and test direction_to_fisheye_lens_polynomial

Every time I add commits and push them I get this message from git:

alexander@cadence:~/rabbit/blender$ git push
Enumerating objects: 17, done.
Counting objects: 100% (17/17), done.
Delt…
Alexander Brock pushed to test-flepo at Alexander-Brock/blender 2024-06-27 17:44:33 +02:00
198703781c Test scale in the manual test for fisheye lens polynomial
60c55a10f1 Add 45° test case for fisheye lens polynomial
Compare 2 commits »
Alexander Brock pushed to test-flepo at Alexander-Brock/blender 2024-06-27 16:04:37 +02:00
3aba593a0b Test symmetry and special values for fisheye_lens_polynomial_to_direction
Alexander Brock commented on pull request blender/blender#123737 2024-06-27 16:03:18 +02:00
Fix and test direction_to_fisheye_lens_polynomial

@Sergey I added 2 test cases which test symmetry and hand-crafted values for fisheye_lens_polynomial_to_direction, could you re-run the test?

Alexander Brock commented on pull request blender/blender#123737 2024-06-26 21:24:13 +02:00
Fix and test direction_to_fisheye_lens_polynomial

Interesting, the test fails on Windows. For the y coordinate the error is ~2.7e-6 which might be some kind of difference in handling of float operations, but for the x coordinate it is ~1.5e-4…

Alexander Brock pushed to test-flepo at Alexander-Brock/blender 2024-06-26 20:46:13 +02:00
dd90d1b2e0 Print more debug output if a test fails
Alexander Brock commented on pull request blender/blender#123737 2024-06-26 16:11:51 +02:00
Fix and test direction_to_fisheye_lens_polynomial

General questions:

  1. When fixing small stuff like float suffix, constness, names, etc. should I amend the previous commit and force-push it?
  2. Should I click "Resolve conversation" when I…
Alexander Brock pushed to test-flepo at Alexander-Brock/blender 2024-06-26 16:05:07 +02:00
380f9f351a Use suffix f for float constants
c15a44a6c5 Make conversion from degree to radians clearer
Compare 2 commits »
Alexander Brock commented on pull request blender/blender#123737 2024-06-26 16:04:20 +02:00
Fix and test direction_to_fisheye_lens_polynomial

I changed it. Usually I prefer the assertions, because I often have hundreds of (pseudo-randomly generated) tests so when I introduce a sign error or similar I get thousands of lines telling me my…

Alexander Brock commented on pull request blender/blender#123737 2024-06-26 16:02:06 +02:00
Fix and test direction_to_fisheye_lens_polynomial

I made everything as const as possible now, the reason I didn't in the first place was that I change k2 in the test-loop because it's zero in all my test cases.