Re: fiologparser.py

[Martin Steigerwald <ms@teamix.de>]

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