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

Center circle detection + calibration integration #1508

Draft
wants to merge 48 commits into
base: main
Choose a base branch
from

Conversation

tuxbotix
Copy link
Contributor

Why? What?

As the title says, should supercede #1057

Fixes #

ToDo / Known Issues

If this is a WIP describe which problems are to be fixed.

Ideas for Next Iterations (Not This PR)

If there are some improvements that could be done in a next iteration, describe them here.

How to Test

Describe how to test your changes. (For the reviewer)

@tuxbotix tuxbotix self-assigned this Nov 20, 2024
@tuxbotix tuxbotix force-pushed the 2024-09_complete_camera_calibration_integration branch from 5923456 to 401b907 Compare November 21, 2024 23:19
Minimum distance between three points of circle ransac is enforced

WIP filtering

Remove unused components and have a more "functional" interface.

Additional outputs are ready
Add cycle instance to the additional outputs

Initial work to get line detection debug images into twix

Can visualise the intermediate stages of line detection via twix.

Get some crude timing

Resize image for easier debug
Fixed the resizing

Parameters for fast debugging

Faster processing setup

View filtered calibration lines
Filter points by horizon
Make the cycler to work

WIP tuning
Cleaning up & clippifying

Refactored the cycler

Render edge points

Use image segments as an alternative source of edge points

updated score calculation scheme and random generator change to ChaCha

Sub-sampling for faster search (= more iterations per search can be done)
Edge detection crate
Downgrade stuff & fix transitive dependency issue
Rename parameters and struct members.
Update the twix panel to work properly

Clippy fixes
@tuxbotix tuxbotix force-pushed the 2024-09_complete_camera_calibration_integration branch from 401b907 to 4981fc8 Compare December 12, 2024 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

1 participant