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

Misc vision stuff #5

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

Misc vision stuff #5

wants to merge 14 commits into from

Conversation

daveoprea
Copy link
Contributor

No description provided.

MisterFancyBananaMan and others added 14 commits February 20, 2019 18:41
i guess the "edited oprea vi" never saved the code which is why you, oprea, were confused as to why there werent any changes in the vi. i fixed it and now i implemented it into the "e - multiple onscreen targets..." dashboard code
this is the frame and fill overlay idea. i think it''ll work ez once i get a base image to work off of
- Fixed bug created when number of target choices was reduced (enum constant had been used instead of enum global variable in "Recognize correct target fixed.vi".
- Removed photo-taking code
- Reduced size of Vision Assistance icon
i havent found the optimal position yet, but now i have the tiny crosshair to help align the dot even better
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.

2 participants