Re: fiologparser.py

[Ben England <bengland@redhat.com>]

no objections here, 

There is some additional documentation in the parse_args() routine in the "help" keyword parameter to parser.add_argument.

  parser.add_argument("FILE", help="collectl log output files to parse", nargs="+")

it's really a fio-generated latency log, not a collectl log

not aware of any dependencies except python-scipy.  It doesn't work with python3 yet.

Additional documentation for the -A option is in:

https://github.com/axboe/fio/pull/177

-ben

----- Original Message -----
> From: "Martin Steigerwald" <ms@teamix.de>
> To: fio@vger.kernel.org
> Cc: "Mark Nelson" <mnelson@redhat.com>, "Ben England" <bengland@redhat.com>, "Jens Axboe" <axboe@kernel.dk>
> Sent: Tuesday, May 24, 2016 6:35:54 AM
> Subject: fiologparser.py
> 
> Hello Mark, Ben,
> 
> I found fiologparser.py in fio 2.10 and for now packaged it into the /usr/
> share/doc/fio. Yet I´d like to more promintly place it in /usr/bin or so… for
> that I would need it to have no script name ending (as according to Debian
> Policy). Would be fine with having it renamed to just  fiologparser? I can
> provide a patch to Jens.
> 
> Also it has no manpage, but a short intro in the script source itself. Do you
> intent to provide a manpage? Otherwise I may have a go at it with help2man or
> so once in a while. It appears to have quite some options:
> 
> # ./fiologparser.py
> usage: fiologparser.py [-h] [-i INTERVAL] [-d DIVISOR] [-f] [-A] [-a] [-s]
>                        FILE [FILE ...]
> fiologparser.py: error: too few arguments
> 
> Care to elaborate what these are doing (besides what is mentioned in script
> header)?
> 
> It requires python-scipy it seems. Anything else?
> 
> Thank you,
> 
>