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

Update the EMS fw to handle the new BMS messages - STINT 1 #508

Closed
valegagge opened this issue Sep 2, 2024 · 5 comments
Closed

Update the EMS fw to handle the new BMS messages - STINT 1 #508

valegagge opened this issue Sep 2, 2024 · 5 comments
Assignees

Comments

@valegagge
Copy link
Member

We want to provide the basic information (advanced alarms and % of charge) the new BMS provides to the user without changing the ETH protocol.

DoD
The EMS fw is updated and verified.

@valegagge
Copy link
Member Author

Regarding the % of charge, this information is carried out in the message 0x620 that already belongs to the protocol, so I didn't need to develop something.
Instead, to provide the alarms to the user I updated the EMS fw to carry this info in one of the fillers of the eOas_battery_status_t.

In addition, I developed a new functionality that lets the user know the cause of the last run. In detail, when the EMS starts, it asks for the last logs to the BMS and prints them as debug messages.

@valegagge
Copy link
Member Author

Along with @MSECode, we validated the first version of this code, and it seems to work as expected.

I still need to improve the code.

@valegagge
Copy link
Member Author

the updated code is here: https://github.com/valegagge/icub-firmware/tree/feat/BMS_new_msg. EMS ver 194

@valegagge
Copy link
Member Author

I complete the first version. see branches:

These last changes are simple aesthetic improvements.

Now the FW handles the new getLog message and prints a debug error when the BMS gets an error in the last run

@valegagge
Copy link
Member Author

The fw is running on the ergocub SN002 as expected. (for IROS conference).

We satisfied the DOD, eve if the task is not completed.
I close this issue in favor of #504

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

No branches or pull requests

2 participants