-
Notifications
You must be signed in to change notification settings - Fork 240
Create C++ Apriltag example #794
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
Conversation
d62ea86
to
72c04b3
Compare
I can't leave a comment but why did the libphotoncam.so change? WPILib version update? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good with two major caveats:
- I'm bad at c++
- not run on a robot.
Plus a few nits about things that might be better to be ignored, at least for example purposes.
But, in terms of communicating the key concepts, it works.
Sim support soon hopefully :). But likely not till after the season.
@@ -0,0 +1,102 @@ | |||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also maybe gitignore?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I vote no, so I can set up WASD bindings for driving around
Update .gitignore
WPILib version update is coz we forgot to bump gradlerio |
Solid |
I just copied ramsete from wpilib
Start PhotonCameraWrapper
bump wpilib everywhere