-
Notifications
You must be signed in to change notification settings - Fork 400
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
feat: Add INA219 battery sensor #2380
feat: Add INA219 battery sensor #2380
Conversation
Future3/develop
This reverts commit 48dd1bf.
Hi, thanks for this addition! We will look into it and get back to you shortly. |
src/jukebox/components/battery_monitor/batt_mon_i2c_ina219/__init__.py
Outdated
Show resolved
Hide resolved
src/jukebox/components/battery_monitor/batt_mon_i2c_ina219/__init__.py
Outdated
Show resolved
Hide resolved
…nit__.py Co-authored-by: s-martin <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this addition. I am not super familiar which battery to chose so I always use a consumer battery pack, but then I am lacking the battery monitor. I'd love to see your set up and types of battery used and how everything is assembled. I guess this could also be great in the documentation
src/jukebox/components/battery_monitor/batt_mon_i2c_ina219/__init__.py
Outdated
Show resolved
Hide resolved
src/jukebox/components/battery_monitor/batt_mon_i2c_ina219/__init__.py
Outdated
Show resolved
Hide resolved
src/jukebox/components/battery_monitor/batt_mon_i2c_ina219/__init__.py
Outdated
Show resolved
Hide resolved
I think this battery monitor can be used with consumer battery packs. The ADC measures the voltage of the battery. If you have access to the raw voltage of the battery, then the battery can be monitored. |
src/jukebox/components/battery_monitor/batt_mon_i2c_ina219/__init__.py
Outdated
Show resolved
Hide resolved
I have a simple USB cable from my battery pack to the Raspi. If I cut the cable and put the sensor in between, this should work it seems. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I haven't been able to test this as I am missing the device.
Code looks good though!
Co-authored-by: s-martin <[email protected]>
Co-authored-by: s-martin <[email protected]>
Thanks for your contribution! 🎉 |
Add an additional battery sensor: The INA2019 is a very commonly used battery sensor. This allows to measure the battery voltage and shut down the PRI Jukebox on low battery.