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

Add GearsBot Constants #6503

Closed
wants to merge 40 commits into from
Closed

Add GearsBot Constants #6503

wants to merge 40 commits into from

Conversation

bakedPotatoLord
Copy link
Contributor

@bakedPotatoLord bakedPotatoLord commented Apr 8, 2024 •
edited by calcmogul
Loading

Adds constants to the cpp gearsbot example, and adds some missed constants to the java gearsbot example.

Fixes #1948.

= and others added 29 commits April 1, 2023 11:49
create drive constants
71ba330
implement drive constants
5b06887
create claw constants
dccb9ba
implement claw constants
6d8b5c8
create wrist constants
93471ef
switch to wrist constants
607aab1
create elevator constants
bbfec15
implement elevator constants and switch to ternary
d1a5c38
create auto constants
d14ae51
implement auto constants
e98788d
create driveStraight Constants
4337d05
implement driveStraight constants
bbedd54
reformat super contructor params
ed1f2c2
create position classes
17022ca
implement constants
6512a74
revert to commands.parallel()
b5e575e
added wrist setpoint constant
4c1e9f3
added missed constant
876cd65
revert to if-statements
048b96a
javaformat
f68a4a4
apply wpiformat
e71a27c
rename constants to kCamelCase
2b67770
import driveStraightConstants directly
708a1dc
import elevatorConstants directly
60d840c
@bakedPotatoLord
add all constants
195d997
@bakedPotatoLord
add subsystem constants
6de6ac9
@bakedPotatoLord
add command constants
252bfbd
@bakedPotatoLord
add missing constants
1db35cb
@bakedPotatoLord
wpiformat
59a6d00
@bakedPotatoLord bakedPotatoLord requested a review from a team as a code owner April 8, 2024 00:04
@bakedPotatoLord bakedPotatoLord requested a review from a team as a code owner April 8, 2024 00:04
@bakedPotatoLord bakedPotatoLord marked this pull request as draft April 8, 2024 16:36
@bakedPotatoLord bakedPotatoLord marked this pull request as ready for review April 8, 2024 17:44
spacey-sooty
spacey-sooty suggested changes Sep 12, 2024
View reviewed changes
Copy link
Contributor

@spacey-sooty spacey-sooty left a comment

Choose a reason for hiding this comment

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

Has conflicts

@Gold856 Gold856 mentioned this pull request Sep 17, 2024
Merged
@bakedPotatoLord bakedPotatoLord marked this pull request as draft September 28, 2024 01:19
@bakedPotatoLord bakedPotatoLord marked this pull request as ready for review September 29, 2024 02:35
@bakedPotatoLord
Copy link
Contributor Author

@spacey-sooty Should be up to date now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@spacey-sooty spacey-sooty spacey-sooty requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Constants class missing in examples
2 participants
@bakedPotatoLord @spacey-sooty