You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As the day rolls over, the daily energy can be incorrect for the previous day. See discussion here.
I can show that it is intermittent based on the data here. This should mean that it is either an issue with the sampling rate of this library or when the Daikin unit internally samples the data.
The fix could be similar to how _compute_diff_energy takes into account the previous time slice (day) to see if there is fractional energy that would be otherwise missing.
The text was updated successfully, but these errors were encountered:
For discussion, I thought it may be interesting to log the unit's time when a data sample was returned. That is, add a list alongside _last_update_by_resource called _last_unittime_by_resource that would be set to the value of cur from the common/get_datetime query.
This is for cases where the unit is set to a different time zone as the client and therefore you could not use the local time to see when a time boundary had been crossed (e.g. when one day switches to the next). Without this extra context, as _compute_diff_energy shows, there is ambiguity for when the extra energy should be added. For example, you could accidentally add the extra energy multiple times in the case where the energy is equal and has not increased across several queries for the data. The code does not seem to know if it had already added yesterday's discrepancy.
As the day rolls over, the daily energy can be incorrect for the previous day. See discussion here.
I can show that it is intermittent based on the data here. This should mean that it is either an issue with the sampling rate of this library or when the Daikin unit internally samples the data.
The fix could be similar to how _compute_diff_energy takes into account the previous time slice (day) to see if there is fractional energy that would be otherwise missing.
The text was updated successfully, but these errors were encountered: