Skip to content
This repository has been archived by the owner on Nov 4, 2022. It is now read-only.

Vision and Autonomous: Boilerplate FRCVision #22

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

Conversation

KhanSimeoni
Copy link
Contributor

Inital code for the FRC vision system.

@KhanSimeoni KhanSimeoni self-assigned this Feb 12, 2022
@KhanSimeoni KhanSimeoni added the new subsystem Entirely new subsytem or rewrite/large addon to existing subsystem. label Feb 12, 2022
Copy link
Member

@KenwoodFox KenwoodFox left a comment

Choose a reason for hiding this comment

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

Id like to see more with networktables before this gets merged.

Kredcool and others added 5 commits February 12, 2022 16:19
Co-authored-by: KenwoodFox <[email protected]>
This has some issues we need to work out.
This adds some additional print statements and a test button command to
make testing a bit easier.
@KenwoodFox KenwoodFox changed the title FRC Vision Vision and Autonomous: Boilerplate FRCVision Feb 13, 2022
Copy link
Member

@KenwoodFox KenwoodFox left a comment

Choose a reason for hiding this comment

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

Sorta getting there.

@KenwoodFox
Copy link
Member

Ill fix this soon

Copy link
Member

@KenwoodFox KenwoodFox left a comment

Choose a reason for hiding this comment

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

@KhanSimeoni i saw you changing all that code, did you push it up?

@KenwoodFox KenwoodFox added the pending test might be solved but it needs to be tested. label Mar 2, 2022
@KenwoodFox KenwoodFox marked this pull request as ready for review March 2, 2022 21:34
Comment on lines +104 to +107
commands2.button.JoystickButton(self.driverController, 5).whenPressed(
LimeAuto(self.drivetrain)
)

Copy link
Member

Choose a reason for hiding this comment

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

This looks good :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
new subsystem Entirely new subsytem or rewrite/large addon to existing subsystem. pending test might be solved but it needs to be tested.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants