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

[Feature Request]: Option to disable charging status or swap current flow direction - INAxxx sensor #5863

Open
hawkeyes0v0 opened this issue Jan 16, 2025 · 1 comment
Labels
enhancement New feature or request requires-protos A change in device that requires changes to protobufs

Comments

@hawkeyes0v0
Copy link

hawkeyes0v0 commented Jan 16, 2025

Platform

NRF52, ESP32, RP2040, Linux Native, Cross-Platform, other

Description

I would like the option to be able to disable 'charging' status when using INA3221, INA219, etc, to sense battery voltage and current flow.

Even better would be the option to reverse current flow direction that activates charging status. WITHOUT a custom firmware build, which I have to currently do.

Currently with the new 2.5.18 firmware version:
Negative current flow measured in channel 1 means that 'charging' status is activated.
Positive current flow measured in channel 1 means that 'charging' status is deactivated.

This is a problem for me since all my remote nodes are hardwired to show negative current flow when discharging. The main issue is that not everyone will have their nodes wired the same way, so being able to control the status would be a nice option instead of having to disassemble and rewire nodes in the field if you want to use official builds.

Related merged feature: #5271

@hawkeyes0v0 hawkeyes0v0 added the enhancement New feature or request label Jan 16, 2025
@b8b8
Copy link

b8b8 commented Jan 16, 2025

Makes sense to me. Negative number is the battery losing energy, positive is its gaining.
It also means when everything is "good" and working, the solar panel, the MPPT output, and the battery will all have positive current when its sunny out.

@fifieldt fifieldt added the requires-protos A change in device that requires changes to protobufs label Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request requires-protos A change in device that requires changes to protobufs
Projects
None yet
Development

No branches or pull requests

3 participants