-
Notifications
You must be signed in to change notification settings - Fork 22
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
Implement entity schema migrations #189
Comments
I think we should do this soon: people keep being confused by the integration sensors, which is entirely avoidable if we read the totals directly from the inverter |
For consistency, where you have the inverter_Totals, consider using these as the source for the utility_meters so the _daily values track the _Today values shown in the app. |
I'm not sure I follow I'm afraid. Why have a utility meter when you have the _Today registers? |
As I understand it (may be wrong...), because:
|
I don't think there's any difference? They're both
We already give people polyfills in some cases. Again it's still a sensor -- it doesn't matter to HA whether the value is being provided by our subclass of
HA tracks long-term statistics for any sensor with the right state_class: https://developers.home-assistant.io/docs/core/entity/sensor/#long-term-statistics
We can read all of these sensors in a single read anyway, for almost 0 extra cost. |
The aim here:
I think the best way to do this is:
LATEST
refers to the latest version.The text was updated successfully, but these errors were encountered: