Spread data spikes over missing days in counter graphs #6472
Conversation
|
For some reason this isn't working on my production box even though it was working in testing on the same type of meter device. |
|
Ah, I figured it out. My test server had gaps where Domoticz itself hadn't been running. My production server had been running but the meter data hadn't been coming in, so there were actual zeroes in the data and not just gaps: I'm confused by the presence of zeroes; surely we shouldn't be writing to We can tweak the graph code to treat those like empty data, but I'm less sure that we should. What do you think? |
When a device hasn't updated within the timeout period, UpdateMeter() already prevents entries from being added to the Meter table. The AddCalendarUpdateMeter() function should leave gaps in Meter_Calendar rather than explicitly inserting zeros, allowing the graph smoothing code to handle the missing data appropriately.
When there are gaps in the data (e.g., device offline for multiple days), the accumulated usage appears as a large spike on the day data resumes. Detect gaps and spread the accumulated value evenly across the missing days to prevent spikes from affecting graph scale. The fillMissingDays function is implemented in the shared counterLogSeriesSupplier module and used by all counter/meter device types (energy-used, energy-generated, gas, water, counter, P1 energy).
|
Fixed the code which introduced the zeroes. We deliberately don't introduce explicit zeroes in the |
a0a55cd to
e57a1b6
Compare
When there are gaps in the data (e.g., device offline for multiple days), the accumulated usage appears as a large spike on the day data resumes. This preprocessing function detects gaps and spreads the accumulated value evenly across the missing days, preventing spikes from affecting graph scale.
It used to look like this:
... now it looks like this: