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

Stereo calibration #26

Closed
wants to merge 16 commits into from
Closed

Stereo calibration #26

wants to merge 16 commits into from

Conversation

dfbakin
Copy link
Member

@dfbakin dfbakin commented Mar 28, 2024

Test

Main changes

  • Changed the approach to saving detetction (vector of points -> map of timstamp:unique_ptr<T*>)
  • Added stereo calibration with synchronisation algorithm to determine, which detections were simultanious

Current test result

Huge reprojection error of more than a thousand from 4 frames, but seemingly right corner detection and object point intersection.

Warning

#24 should be merged prior to this PR, because it defines CameraIntrinsicParameters implementation

UPD

PR was closed due to temporary migration from ROS2 and real-time algothithms to a consequtive pipeline (with separate binary executables for each stage).

@dfbakin dfbakin added the enhancement New feature or request label Mar 28, 2024
@dfbakin dfbakin self-assigned this Mar 28, 2024
@dfbakin dfbakin force-pushed the stereo-calibration branch from 0bfbcee to d39defd Compare April 1, 2024 23:13
@dfbakin dfbakin requested a review from dasimagin April 6, 2024 00:27
@dfbakin dfbakin marked this pull request as ready for review April 6, 2024 00:27
@dfbakin dfbakin marked this pull request as draft April 7, 2024 17:56
@dfbakin dfbakin removed the request for review from dasimagin May 2, 2024 16:53
@dfbakin dfbakin closed this May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

Successfully merging this pull request may close these issues.

1 participant