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

Safe Limits API (#1325) #1355

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

unaiur
Copy link

@unaiur unaiur commented Sep 18, 2023

This is a proposal for feature request #1325.

When an inverter has a safe limit timeout enabled, a new /api/limit/safe endpoint is enabled to enact limits safely.

The caller needs to ensure that the limit is confirmed before the timeout expires; otherwise, the limit is rolled back to the safe limit value.

@unaiur unaiur force-pushed the safe-limits-v1 branch 2 times, most recently from 282ea23 to abe69b3 Compare September 18, 2023 19:49
When an inverter has the safe limit timeout enabled, a new /api/limit/safe WebApi
is enabled that allow to setup dynamic non-persistent limits.

Those dynamic limits are reverted if they are not confirmed periodically before the
safe limit timeout expires.

This way, if the controller or the WiFi router dies, the limit is safely restored
to the default limit.
@RobertLukan
Copy link

This is a very good feature. Do you think it will get merged ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants