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

Finalize VCU Infrastructure #39

Merged
merged 58 commits into from
Mar 3, 2023
Merged

Conversation

rafguevara14
Copy link
Collaborator

@rafguevara14 rafguevara14 commented Feb 23, 2023

Summary

This should be the last big PR for this repo. It adds many generic implementations of tasks and functions to support future features.

An accompanying design document will be added or a meeting should be held to discuss the design and implementation of the new architecture and modules before this is merged.

Notable Files

Use this PR File Filter to help narrow down things and remove the misc project and deleted files.

There are a lot of changes made in this PR. Here are the most important of folders/files that need review:

Major functional changes:

  • Finalized FreeRTOS priority hierarchy
  • Standardized logging
    • Logger task
  • Generic event handler task
    • State machine and ready-to-drive procedure
    • Inject events to state machine for testing
  • Fault timers reenabled and fixed

Cleanup

  • Add diagrams
  • Remove all irrelevant tasks and modules
  • Add VCU common functions and constants

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.

Looks good. May want to remove drawio code file later but for now, it can stay.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants