Skip to content

Fixes#149

Open
KSMehta11 wants to merge 27 commits into
developfrom
fixes
Open

Fixes#149
KSMehta11 wants to merge 27 commits into
developfrom
fixes

Conversation

@KSMehta11

@KSMehta11 KSMehta11 commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Changes

  • Fixed state machine fault evaluation, transitions, charging, and balancing behavior.
  • Fixed DCL/CCL CAN flickering, updated P50B configurations, and corrected charge-current fault handling.
  • Improved cell temperature sanitization and mutex protection.
  • Added sanitizer unit tests and updated state machine, DCL, and CCL tests.
  • Ported HV plate DTI handling, DTI/charger CAN parsing, and precharge updates from develop-dti.
  • Updated generated CAN code and supporting driver dependencies.

To Do

  • Will merge the open-wire detection changes in a separate PR with the ticket linked and the required CAN messages.
  • Will also add CAN reporting for per-cell S-ADC voltages.

Test Cases

  • Tested using the Simulink-based HIL tester.

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.

  • No merge conflicts
  • All checks passing
  • Remove any non-applicable sections of this template
  • Assign the PR to yourself
  • Request reviewers & ping on Slack

@KSMehta11 KSMehta11 self-assigned this Jul 6, 2026
@KSMehta11
KSMehta11 marked this pull request as ready for review July 8, 2026 22:18
@KSMehta11
KSMehta11 requested review from caiodasilva2005 and jr1221 and removed request for caiodasilva2005 and jr1221 July 8, 2026 22:19
@KSMehta11
KSMehta11 marked this pull request as draft July 9, 2026 18:21
@KSMehta11
KSMehta11 marked this pull request as ready for review July 13, 2026 22:41
Comment thread Core/Src/hv_plate.c Outdated
Comment thread Core/Src/state_machine.c Outdated
Comment thread Core/Src/state_machine.c
Comment thread Core/Src/state_machine.c Outdated
Comment thread Core/Src/state_machine.c Outdated
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.

2 participants