Skip to content

Move constants from Drive elsewhere#18

Closed
onlycs wants to merge 11 commits intomasterfrom
constant-move
Closed

Move constants from Drive elsewhere#18
onlycs wants to merge 11 commits intomasterfrom
constant-move

Conversation

@onlycs
Copy link
Contributor

@onlycs onlycs commented Feb 5, 2026

Specifically, constants that don't belong there, including but not limited to:

  • mass
  • MOI
  • wheel friction
  • speed limits (will be used by, e.g., autos and repulsor)
  • physical constants such as length, width, etc

this PR builds off of #16 and #17, just to avoid future merge conflicts and stuff

as a bonus, the new constants are much more well-documented than the old ones.

@stevemurphymsu
Copy link
Member

These changes are rejected. Please do not change the location of constants from the advantage kit template.

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.

3 participants