On Dienstag, 24. Mai 2016 21:04:13 CEST Mark Nelson wrote: > On 05/24/2016 03:47 PM, Ben England wrote: > > Mark, I didn't notice the sample weighting code before. Weighting of > > samples might work for averaging, but it doesn't work for percentiles, > > min or max provided by -A option. I guess for min this won't be an issue > > generally, since min-latency samples will probably fall entirely within a > > time interval. But for max or higher percentiles it will *definitely* be > > an issue. For example, a really high latency sample could be the max > > for a whole range of time intervals. > I went back and reworked the print and per-interval functions so that > they are part of a Printer class and Interval class respectively. It > cleaned the code up pretty nicely. I was also able to integrate the > "-A" code to use a lot of the existing statistics and formatting code. > It now supports the "-d" flag for example. While at it, Mark, could you also do the renaming to fiologparser? Otherwise I´d prepare a patch, but then we risk a trivial merge conflict :) Thanks, Martin -- To unsubscribe from this list: send the line "unsubscribe fio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
