8000 Create C++ Apriltag example by mcm001 · Pull Request #794 · PhotonVision/photonvision · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

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

Merged
merged 5 commits into from
Feb 12, 2023
Merged

Conversation

mcm001
Copy link
Contributor
@mcm001 mcm001 commented Feb 10, 2023

I just copied ramsete from wpilib

Start PhotonCameraWrapper

bump wpilib everywhere

@mcm001 mcm001 requested a review from a team as a code owner February 10, 2023 14:03
@mcm001 mcm001 force-pushed the apriltag-c++-example branch from d62ea86 to 72c04b3 Compare February 10, 2023 14:04
@gerth2
Copy link
Contributor
gerth2 commented Feb 10, 2023

I can't leave a comment but why did the libphotoncam.so change? WPILib version update?

gerth2
gerth2 previously approved these changes Feb 10, 2023
Copy link
Contributor
@gerth2 gerth2 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 with two major caveats:

  1. I'm bad at c++
  2. 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 @@
{
Copy link
Contributor

Choose a reason for hiding this comment

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

Also maybe gitignore?

Copy link
Contributor Author

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
@mcm001
Copy link
Contributor Author
mcm001 commented Feb 11, 2023

WPILib version update is coz we forgot to bump gradlerio

@mcm001 mcm001 closed this Feb 11, 2023
@mcm001 mcm001 reopened this Feb 12, 2023
@gerth2
Copy link
Contributor
gerth2 commented Feb 12, 2023

Solid

@gerth2 gerth2 merged commit a2dfe48 into PhotonVision:master Feb 12, 2023
@mcm001 mcm001 deleted the apriltag-c++-example branch February 12, 2023 05:12
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
0