Skip to content
This repository has been archived by the owner on Jan 2, 2024. It is now read-only.

Nav 260 - BMS Power Management System #146

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Nav 260 - BMS Power Management System #146

wants to merge 4 commits into from

Conversation

hhenry01
Copy link
Member

@hhenry01 hhenry01 commented Aug 10, 2022

This PR implements a battery power manager for Raye such that the power load is distributed evenly across the port and starboard batteries.

@MoizLokh I've been closely following the design document on Confluence. So far only 3 unit tests are implemented and passing.

Edit: Confluence page with the design and test plan: https://ubcsailbot.atlassian.net/l/cp/cPJg1WYj

@hhenry01 hhenry01 requested a review from MoizLokh August 10, 2022 06:53
@hhenry01
Copy link
Member Author

@bryanzhang2002 @andchirik @Kerem-Oktay @derain12 This is the code for the proposed battery management system.

@derain12
Copy link

@hhenry01 please add reference to Confluence page here for review purposes

@hhenry01
Copy link
Member Author

@hhenry01
Copy link
Member Author

All unit tests outlined on the Confluence page are passing. If the implementation meets ELEC's expectations then it is ready to test with the physical batteries.

@hhenry01 hhenry01 marked this pull request as ready for review August 12, 2022 06:04
CAN commands to controller the battery are continuously sent at set
intervals.
@hhenry01 hhenry01 closed this Nov 19, 2022
@hhenry01 hhenry01 reopened this Nov 19, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants