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

Compensate for turret position when looking for clusters #133

Merged
merged 1 commit into from
Feb 1, 2025

Conversation

james-ward
Copy link
Contributor

No description provided.

@LucienMorey
Copy link
Contributor

This looks right to me because you're just shifting one side of the line you draw to the camera base. I will sim and check a little more ahead of time after work, but tentatively LGTM

@james-ward
Copy link
Contributor Author

This looks right to me because you're just shifting one side of the line you draw to the camera base. I will sim and check a little more ahead of time after work, but tentatively LGTM

I messed around by setting the transform of the camera to really offset values and rotations, and by limiting the motion of the servo. Then you can see it is now pointing in the right direction. Single tags are a good way to verify this with the big offsets because it is very obvious.

@james-ward
Copy link
Contributor Author

Closes #125

Copy link
Contributor

@CalvinOdropbear CalvinOdropbear left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me

@LucienMorey LucienMorey force-pushed the camera_pos_compensation branch from f1433a1 to 4e2a518 Compare February 1, 2025 00:24
@LucienMorey
Copy link
Contributor

Tested on robot and things seem to be all clear

@LucienMorey LucienMorey enabled auto-merge February 1, 2025 00:25
@LucienMorey LucienMorey merged commit 52bb3b3 into main Feb 1, 2025
5 checks passed
@LucienMorey LucienMorey deleted the camera_pos_compensation branch February 1, 2025 00:25
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.

3 participants