Closed Jonathan-Gore closed 7 months ago
Thanks for the PR @Jonathan-Gore ! See the "Build" section from the README to compile and test the changes.
On it
Updated k -> k1 and successfully built the application!
Ran the newly built .exe on the banana dataset with both PINHOLE and SIMPLE_RADIAL camera models and both were successful! simple_radial_splat.zip
Fantastic, thanks for testing and the useful addition! 🙏
Added the SimpleRadial camera model to colmap.cpp from https://github.com/colmap/colmap/blob/0ea2d5ceee1360bba427b2ef61f1351e59a46f91/src/colmap/sensor/models.h#L235
Copied syntax found in SimplePinhole, copying fy=fx for the models with only one focal length parameter, like SimpleRadial.
I'm new to c++ so I'm not 100% sure how to test the code.