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

Improve the status handler for BAT and BMS #952

Open
valegagge opened this issue Mar 18, 2024 · 4 comments
Open

Improve the status handler for BAT and BMS #952

valegagge opened this issue Mar 18, 2024 · 4 comments
Assignees

Comments

@valegagge
Copy link
Member

Is your feature request related to a problem?

Improve the status information related to the BMS and BAT. Currently, both objects rely on the embObjBattery device which is not at all compliant with the BAT board.

Even if the best solution is to create an ad-hoc device and expose ad-hoc API in a new interface, I think that the first step is to improve the returned status.
Currently, the status is not related to the enum exposed in the API interface but a number that is a bitmask of the status related to BAT or BMS. In this way the user cannot use the status the information.

The solution you would like to have available

The idea here is to map the status of BAT and BMS to a significant value in the enum provided by the API and update the documentation accordingly.

image

Alternatives you have considered

No response

Additional context

No response

@valegagge
Copy link
Member Author

cc @MSECode

@pattacini
Copy link
Member

Hi @MSECode @valegagge

Is this related to #953?

@MSECode
Copy link
Contributor

MSECode commented Mar 26, 2024

Not really, it is related to further improvements we wanna port to the management of the status message for both BAT and BMS and their relation with the yarp interface iBattery, since now there are misalignments between how yarp defines the Status of the battery and how the flags regarding status and errors are raised by bms and bat.

@valegagge
Copy link
Member Author

We are waiting for the new BMS to finalize the work.

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

3 participants