-
-
Notifications
You must be signed in to change notification settings - Fork 46
listen to gridReverse-state to rely on bypass or internal logic #907
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
base: master
Are you sure you want to change the base?
Conversation
| # grid_state, | ||
| # ) | ||
| # Treat missing gridReverse as allowed; otherwise rely on its string state ("disabled", "allow", "forbidden"). | ||
| if d.state == DeviceState.SOCFULL and (grid_state is None or grid_state == "allow"): |
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.
Do all devices have the gridReverse entity?
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.
No, the Hub-family i.e. have a entity passMode for that.
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.
Ok, so the additional check should be fine, then. With the recent updates, I'm not sure if thats easily mergeable - however, the code usually seems to assume that any power generated by the solar panels is passed through, see the comment:
// battery in bypass, should automatically supply the remainging solar power to the house
This isnt true for the Hyper 2000, unfortunately. This only works when gridReverse is allowed. If it's disabled, it should still be be capped at whatever the p1 meter suggests.
|
Unfortunately, the same issue remains in the major 1.2.1 release. Whenever the battery is full, mode is set to smart_charge and grid reverse is being disabled, solar power provided collapses to 0. |
|
Is this true, that you can control the output power, if gridreverse is disabled (not allowed) and the batterie is full? I can't test it, I use only AC chargeing. As I know, if gridreverse is allowed, you can't avoid, that all the power is feed into the grid, even if you set the output to 0. |
Follow-up of #874, currently untested due to missing weather.
This should, in theory, enable the pass-through on full soc when the grid reverse is enabled - and only then.