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

#23 - state machine skeleton #34

Merged
merged 4 commits into from
Mar 19, 2024
Merged

#23 - state machine skeleton #34

merged 4 commits into from
Mar 19, 2024

Conversation

RChandler234
Copy link
Contributor

@RChandler234 RChandler234 commented Mar 17, 2024

Changes

Copied over the stuff from Cerberus, changed it to only have one state that we track instead of two (drive state and functional state), lmk if I should add that back in. Beyond that, the rest of this one should be adding in all the states/ state transition stuff that we know of and actual adding this functionality into main

To Do

Any remaining things that need to get done

  • Fill in actual states/do a basic version of the state transition table
  • Hook up in main.c somewhere

Checklist

It can be helpful to check the Checks and Files changed tabs.
Please reach out to your Project Lead if anything is unclear.
Please request reviewers and ping on slack only after you've gone through this whole checklist.

  • All commits are tagged with the ticket number
  • No merge conflicts
  • All checks passing
  • Remove any non-applicable sections of this template
  • Assign the PR to yourself
  • Request reviewers & ping on Slack
  • PR is linked to the ticket (fill in the closes line below)

Closes #23 (issue #)

@RChandler234 RChandler234 self-assigned this Mar 17, 2024
Copy link
Contributor

@nwdepatie nwdepatie left a comment

Choose a reason for hiding this comment

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

Approved, looks like it'll work and it is expandable

CM7/Core/Inc/state_machine.h Show resolved Hide resolved
@nwdepatie nwdepatie merged commit c537e5e into main Mar 19, 2024
1 check passed
@nwdepatie nwdepatie deleted the #23-state-machine branch March 19, 2024 00:59
MandarinPine18 pushed a commit that referenced this pull request Mar 22, 2024
* #23 - state machine skeleton

---------

Co-authored-by: RChandler234 <root@DESKTOP-DEAD2D4>
MandarinPine18 pushed a commit that referenced this pull request Mar 29, 2024
* #23 - state machine skeleton

---------

Co-authored-by: RChandler234 <root@DESKTOP-DEAD2D4>
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.

[RTOS] - Create State Machine
2 participants