-
Notifications
You must be signed in to change notification settings - Fork 1
Conversation
resample: add test (currently panics)
}, tp{ | ||
unixTimePointer(10, 0), float64Pointer(1.5), | ||
}, tp{ | ||
unixTimePointer(15, 0), float64Pointer(2), |
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.
Here the value is 2 because the last seen value from the original series is in the 9th second
but I'm not whether we should keep it like this or it should be 1.5 instead (the last value of the resampled series).
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.
So when downsampling, in order to create a regular series there may be gaps because there was no data in the window to aggregate.
For the first pass I think these can be timestamp,null for now. But eventually we will want some "fill" options for how this is handled. If we were to create an irregular series, then we would omit the point entirely.
* Move resample code to separate file. * Declare a separate function for parsing the rule. * Compile regular expression only once
- Rename "VarToReduce" field of "ResampleCommand" to "VarToResample" - Drop dependency to "gonum.org/v1/gonum/stat" - Add support for min, max, count and sum downsampling function
No description provided.