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 battery chemistry in published BatteryState #7

Open
tfoote opened this issue May 4, 2016 · 4 comments
Open

Update battery chemistry in published BatteryState #7

tfoote opened this issue May 4, 2016 · 4 comments

Comments

@tfoote
Copy link
Contributor

tfoote commented May 4, 2016

It's available on most systems and there's a field in the BatterState. http://docs.ros.org/api/sensor_msgs/html/msg/BatteryState.html

This requires #6

And will need to do a mapping from the linux strings to the msg enums.

@stonier
Copy link
Contributor

stonier commented May 11, 2016

  • Merged the pull request into a kinetic branch
  • Old master moved to indigo branch

And will need to do a mapping from the linux strings to the msg enums.

@tfoote What were you thinking about with this?

@stonier
Copy link
Contributor

stonier commented May 11, 2016

Right now I'm getting URLError and SSLError timed out problems with bloom-release. Am guessing this is just a connection problem, so I'll try releasing again this afternoon.

@tfoote
Copy link
Contributor Author

tfoote commented May 11, 2016

This was just a feature enhancement request to populate more of the BatteryState fields. It's extra information, but I did notice that the acpi info seems to have this available so it would be great to add if someone has a free moment.

@stonier
Copy link
Contributor

stonier commented May 11, 2016

Got the idea... 👍

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