Change time interval rounding to 5s

[amosyuen]

Change time interval rounding to 5s which matches iotawatts data log interval. See https://docs.iotawatt.com/en/master/query.html. This allows client to request data more frequently then every 30s.

[llamafilm]

@amosyuen what problem is this attempting to solve? Are you seeing inaccurate data with the 30 second interval? I'm new to this project, having just installed my first IotaWatt, but so far my data looks good.

[mmiller7]

Is there any update on this?

It appears the hard-coded 30 seconds instead of referencing the parameter timespan limits the update rate even when a non-default timespan is specified in the update method. This results in inaccurate data being returned with intermediate values that are not actually correct for the real time power draw.

The IoTaWatt built-in web UI has a 5 second refresh interval on displayed data. It should be possible to have a 5 second refresh interval instead of being confined to a 30 second interval.

When will this be fixed?

[mmiller7]

Actually I think this fix still has a bug in it - 30 seconds worked calculating the difference with an if/else because that is half a minute but I believe the correct way to make a 'diff' would be using mod-operator. I think you wanted diff = seconds % 5

[amosyuen]

@mmiller7 Good catch, fixed the commit. Though not sure if maintainers are ever going to merge this fix in

[mmiller7]

I have low expectations given how long its been open. Frustrating as I think this bug also would affect everything that uses it as a dependency.

Unrelated, the friend I had helping me understand the Python pointed out the one warning that prints out also is missing the f in front of "Nothing to query, update() called too soon, must wait {timespan}" so it prints a string literal instead of the requested timespan variable contents...but at least that doesn't affect the logical functionality of it.