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

Add Throttle Sim Mode #35

Merged
merged 32 commits into from
Feb 3, 2023
Merged

Add Throttle Sim Mode #35

merged 32 commits into from
Feb 3, 2023

Conversation

rafguevara14
Copy link
Collaborator

@rafguevara14 rafguevara14 commented Feb 2, 2023

Summary

This PR adds the ability to receive APPS 1 and 2 values via an encoding scheme through UART as defined here.
https://docs.google.com/document/d/1JWH6s5qf8QeWGzkVzae5GWXK8BNsT8EGFw0WuFVnmiY/edit

This functionality will then be used in conjunction with @kevinl03 's python scripts to fully simulate deal values in real-time and get real throttle output from the VCU. More details can be found in the VCU User Manual...A test file written in python is also added that sends arbitrary values with known VCU responses to test that the VCU is actually processing and outputting data as expected.

TODO:

  • Create VCU User Manual

Misc features

  • New build config in CCS for sim and normal mode
  • All compiler optimizations removed for ease of debugging
  • More useful print statements (tells you when in sim mode, spams terminal if throttle task not initialized properly)

Spinoff Issues

Here are some other features and TODOs added that are outside the scope of this PR. These need to be planned out and prioritized accordingly @kkramer215
#36
#37
#38

@rafguevara14 rafguevara14 linked an issue Feb 2, 2023 that may be closed by this pull request
3 tasks
Copy link
Contributor

@kkramer215 kkramer215 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@kevinl03 kevinl03 left a comment

Choose a reason for hiding this comment

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

LGTM

@kevinl03 kevinl03 merged commit 5e34c69 into develop Feb 3, 2023
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.

4D: VCU Simulation Mode
3 participants