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

[Request] MQTT topic subscribe Battery Level for DPL #293

Closed
niceborn opened this issue Jul 5, 2023 · 9 comments · Fixed by #589
Closed

[Request] MQTT topic subscribe Battery Level for DPL #293

niceborn opened this issue Jul 5, 2023 · 9 comments · Fixed by #589
Assignees
Labels
enhancement New feature or request

Comments

@niceborn
Copy link

niceborn commented Jul 5, 2023

Is your feature request related to a problem? Please describe.

I don't have a Pylontech battery but a Junteks KH-110F bluetooth battery coulometer (http://www.junteks.com/productinfo/977969.html) to get detailed battery information. which can be additional read by a serial adapter. I connected it to a second ESP32 and submit all received data to my MQTT broker.
Included in the information is also the battery level.
Right now only the battery DC voltage in Dynamic Power limiter Settings can be used.

Describe the solution you'd like

I would like to have an option to use an MQTT battery level for the setting in DPL. Setting could be the same as for power meter settings, where also an MQTT option exists.

Describe alternatives you've considered

No response

Additional context

No response

@niceborn niceborn added the enhancement New feature or request label Jul 5, 2023
@madmartin
Copy link

@helgeerbe I think this is a useful enhancement.

I write a PR for this.

@helgeerbe helgeerbe assigned helgeerbe and madmartin and unassigned helgeerbe Jul 17, 2023
@schlimmchen
Copy link
Member

@madmartin I implemented support for the Jikong (JK) BMS as the battery interface. The software is already in use for roughly a week now.

I assume that part of my work could be beneficial to this feature (assuming you have not already implemented that parts that I can contribute). This feature request is asking for a new battery interface, which will use MQTT as the protocol to retrieve supported battery stats rather than CAN/UART/RS485. If you agree, then you might want to base your work on https://github.com/schlimmchen/OpenDTU-OnBattery/commits/additional-battery-interfaces where I already made the respective changes to (1) enable the Battery admin web interface to present a list of supported communication protocols, (2) changed texts so that it is clear that not only Pylontech CAN is supported as battery interface and (3) changed the Pylontech CAN code such that it will only run if it is selected as the battery interface protocol.

I am happy to exchange notes on this topic.

@madmartin
Copy link

@schlimmchen got it... I was just designing the Web UI part, so I take a look at your fork before I continue.

@TheGrumpyBadger
Copy link

I still like to see a MQTT connection for SOC. In my case a Victron SmartShunt is connected to a Raspberry with VenusOS.

@swingstate
Copy link

I'll support this request too. Receiving SOC data as universal MQTT messages would make onBattery even more open to more battery configuration.

I have a pending request open to support the SOC reading through VE.Direct from a Victron Shunt (Since this project has already integrated parts of Victron hardware), however the MQTT approach would be more universal and I suggest to prioritise the MQTT extension over Ve.Direct for SOC data (can come later).

Having worked with my setup for a while now, I am clearly missing the feature that onBattery can use SOC information. using the voltage is maximum an indicator.

@schlimmchen
Copy link
Member

@madmartin #319 is ready for review and merging. Feel free to start implementation of the feature described in this issue and let me know if you need any info on something I wrote for the JK BMS support. I would be happy to see that you find your way around my code easily so that implementing the MQTT battery provider is fun 😊

@88markus88
Copy link

Ist ja schon eine Weile offen .... Ist absehbar daß Input von Batteriedaten via MQTT kommen wird?

@DerWe
Copy link

DerWe commented Nov 23, 2023

I'd support this feature as well. Is there anyone working on this Feature currently? Is it foreseeable to see it in any further release?

schlimmchen added a commit to schlimmchen/OpenDTU-OnBattery that referenced this issue Jan 3, 2024
this battery provider implementation subscribes to a user-configurable
topic to retrieve the battery SoC value. the value is not re-published
under a different topic. there is no card created in the live view,
since the SoC is already part of the totals at the top of the live view.

closes hoylabs#293.
relates to hoylabs#581.
schlimmchen added a commit to schlimmchen/OpenDTU-OnBattery that referenced this issue Jan 3, 2024
this battery provider implementation subscribes to a user-configurable
MQTT topic to retrieve the battery SoC value. the value is not
re-published under a different topic. there is no card created in the
web app's live view, since the SoC is already part of the totals at the
top of the live view. that is the only info this battery provider
implements.

closes hoylabs#293.
relates to hoylabs#581.
schlimmchen added a commit to schlimmchen/OpenDTU-OnBattery that referenced this issue Jan 3, 2024
this battery provider implementation subscribes to a user-configurable
MQTT topic to retrieve the battery SoC value. the value is not
re-published under a different topic. there is no card created in the
web app's live view, since the SoC is already part of the totals at the
top of the live view. that is the only info this battery provider
implements.

closes hoylabs#293.
relates to hoylabs#581.
helgeerbe pushed a commit that referenced this issue Jan 4, 2024
this battery provider implementation subscribes to a user-configurable
MQTT topic to retrieve the battery SoC value. the value is not
re-published under a different topic. there is no card created in the
web app's live view, since the SoC is already part of the totals at the
top of the live view. that is the only info this battery provider
implements.

closes #293.
relates to #581.
Copy link

github-actions bot commented Apr 4, 2024

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new discussion or issue for related concerns.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants