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.

Sign up for GitHub

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

Jump to bottom

Find balls #45

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Find balls #45

wants to merge 14 commits into from

Conversation

CohenAriel
Copy link
Contributor

@CohenAriel CohenAriel commented Feb 7, 2022 •
edited
Loading

resolves #42

@CohenAriel CohenAriel self-assigned this Feb 7, 2022
@CohenAriel CohenAriel changed the title Add getDistance method Find balls Feb 7, 2022
@Starlight220 Starlight220 linked an issue Feb 7, 2022 that may be closed by this pull request
implement vision #42
Closed
@CohenAriel CohenAriel marked this pull request as ready for review February 8, 2022 10:31
@CohenAriel
Copy link
Contributor Author

CohenAriel commented Feb 8, 2022 •
edited
Loading

The actual vision processing is done with a gui for some reason, so I guess I'm done...

This was linked to issues Feb 13, 2022
vision-based cargo pickup #33
Open
trajectory-based intake command #38
Open
@CohenAriel CohenAriel force-pushed the find-balls branch 2 times, most recently from ec3bca4 to d3f32be Compare February 22, 2022 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TapChap TapChap Awaiting requested review from TapChap

@Starlight220 Starlight220 Awaiting requested review from Starlight220

Requested changes must be addressed to merge this pull request.

Assignees

@CohenAriel CohenAriel

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

trajectory-based intake command vision-based cargo pickup implement vision
3 participants
@CohenAriel @Starlight220 @ExcaliburFRC6738