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

Set DPL settings (especially "Upper power limit") via MQTT #724

Closed
Napizaki opened this issue Mar 7, 2024 · 9 comments
Closed

Set DPL settings (especially "Upper power limit") via MQTT #724

Napizaki opened this issue Mar 7, 2024 · 9 comments
Labels
enhancement New feature or request

Comments

@Napizaki
Copy link

Napizaki commented Mar 7, 2024

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

No response

Describe the solution you'd like

In addition to pull request #677, it would be good to be able to influence the "Upper power limit" of the DPL via MQTT to react to certain conditions (weather, load (e.g. electric car), ...).

Describe alternatives you've considered

No response

Additional context

No response

@Napizaki Napizaki added the enhancement New feature or request label Mar 7, 2024
@cerise21
Copy link

It is already possible to modify the DPL upper power limit via POST to the Web API. For instructions see here and there.
However this approach involves rewriting the config every time (which is probably bad in the long run (?)).

Thus I would appreciate it, if a means to modify the upper power limit (without a config rewrite) were implemented.

@schlimmchen
Copy link
Member

@Napizaki @cerise21 Do you consider settings other than the upper power limit to be important enough to be changed using MQTT? Since #677 there are overrides for all SoC and voltage thresholds.

AS every value that can be overridden in the future will have to be maintained forever, I would like to only add MQTT overrides for settings that people actually want to modify for good reasons. In particular, we'll need to define a respective topic, create and maintain the respective documentation, and need to display a line in the to-be-created DPL card of the live view which indicates the override to be active and with what value.

@cerise21
Copy link

cerise21 commented Apr 7, 2024

Up to now the upper power limit is the only DPL value that I (sometimes) want to change a couple of times per day to account for special energy usage scenarios (e. g. increase for temporary high loads when battery full or high irradiation; decrease for highly intermittent loads or at night time).

@Napizaki
Copy link
Author

As described above, I would personally be very interested in changing this value dynamically.
This would allow me, among other things, to charge my electric car with full inverter power as soon as it is plugged in. If it is not plugged in, the inverter power is reduced again to e.g. 600W.
For me personally, this would be a great added value.

@stanpete78
Copy link

stanpete78 commented Apr 19, 2024

As I wrote in a different post, I‘m very happy with my DPL automation via WEB API that reacts to how much solar energy is beeing fed into the system of two (or more) MPPTs and then adapts the upper power limit value every 10 minutes to this value.
I consider the upper power limit the most important value to change with an automation in order to get close to maxing out your system.

So I‘d also be happy to see an easier way of integrating this value in Home Assistant instead of ugly and complicated Web API calls.

@mastertape
Copy link

The ability to set power consumption and hysteresis settings via MQTT/REST API would be a valuable addition.

@schlimmchen
Copy link
Member

Among you who subscribed to this issue: Is there nobody willing to implement this? This is low-hanging fruit 😁 The challenging bits and pieces are all already there, see MqttHandlePowerLimiter.cpp. It's about copy'n'pasting a couple lines of code and testing the changes. As a first step, no new DPL card is required in the live view... This is optional and is a second step.

cerise21 added a commit to cerise21/OpenDTU-OnBattery that referenced this issue Jun 12, 2024
@schlimmchen
Copy link
Member

Completed by #1047 and #1063.

Copy link

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 Jul 22, 2024
@hoylabs hoylabs unlocked this conversation Jul 28, 2024
@hoylabs hoylabs locked as resolved and limited conversation to collaborators Jul 28, 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

No branches or pull requests

5 participants