Skip to content
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

Handle distortion in the ipm #64

Merged
merged 5 commits into from
Mar 23, 2024
Merged

Handle distortion in the ipm #64

merged 5 commits into from
Mar 23, 2024

Conversation

Flova
Copy link
Contributor

@Flova Flova commented Mar 18, 2024

Currently, no distortion is considered in the IPM and we assume all inputs come from a rectified image. But this might not always be the case. Many CV approaches also perform well on unrectified images and skipping the rectification can be good in terms of performance. In cases like this, the distortion needs to be accounted for during the inverse perspective mapping. This pr introduces a dep to OpenCV to calculate the ray directions. This is able to account for the distortion. It is slightly slower compared to the previous approach, but it still only takes ~1/100 seconds to convert a reasonably sized image in its entirety (which is not the case most of the time).

Closes #11

@Flova Flova marked this pull request as draft March 18, 2024 19:53
@Flova Flova marked this pull request as ready for review March 19, 2024 11:51
@Flova
Copy link
Contributor Author

Flova commented Mar 19, 2024

Docs: ros-sports/ipm-docs#5

@Flova Flova merged commit b63672e into rolling Mar 23, 2024
3 of 4 checks passed
@Flova Flova deleted the feature/distortion branch March 23, 2024 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support for distortion matrix
2 participants