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.

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

Format code for prototype/deploy_code_to_brain_old/code_for_testing #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

baysmith
Copy link
Contributor

Formatted code using the style defined in the .clang-format file.

PlatinumWolf128 added a commit that referenced this pull request Jan 8, 2025
- The maximum range of the frontBackVelocity being
  passed into robotDrive is [-1, 1], even though
  the velocity range for robotDrive is [-100,
  100], so now when frontBackVelocity is passed
  into the robotDrive function it is multiplied by
  100
- We realized that controllerFrontBackPosition was
  equal to the position of the joystick on axis
  #3, and that controllerLeftRightPosition was
  assigned axis #4.
- Weirdly, the bot was still
  able to drive with acceleration disabled, but we
  think the axis mix-up made it so the bot
  couldn't turn while acceleration was enabled.
- Something in the arcade drive formula is causing
  the controls to be inverted (e.g. Pushing the
  joystick forwards made the bot drive backwards).
  We weren't able to find the root of the issue
  but know that making controllerFrontBackPosition
  and controllerLeftRightPosition negative solves
  the issue (e.g. Calling robotDrive like
  robotDrive(-controllerFrontBackPosition,
  -controllerLeftRightPosition)).
- Further testing is needed.
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.

1 participant