From: Jens Axboe <axboe@kernel.dk>
To: Ben England <bengland@redhat.com>, Mark Nelson <mark.a.nelson@gmail.com>
Cc: Martin Steigerwald <ms@teamix.de>,
fio@vger.kernel.org, Mark Nelson <mnelson@redhat.com>
Subject: Re: fiologparser.py
Date: Tue, 24 May 2016 09:28:39 -0600 [thread overview]
Message-ID: <57447327.3010503@kernel.dk> (raw)
In-Reply-To: <1549707390.65536830.1464103326492.JavaMail.zimbra@redhat.com>
On 05/24/2016 09:22 AM, Ben England wrote:
>
>
> ----- Original Message -----
>> From: "Mark Nelson" <mark.a.nelson@gmail.com>
>> To: "Ben England" <bengland@redhat.com>, "Martin Steigerwald" <ms@teamix.de>
>> Cc: fio@vger.kernel.org, "Mark Nelson" <mnelson@redhat.com>, "Jens Axboe" <axboe@kernel.dk>
>> Sent: Tuesday, May 24, 2016 10:04:14 AM
>> Subject: Re: fiologparser.py
>>
>> Let's see if we can remove the numpy and scipy dependencies. It looks
>> like we are just using it for min/average/median/max/percentile
>> calculations. It would be nice if users didn't need anything other than
>> argparse.
>>
>
> Just curious, why is scipy a problem? Is it because CBT isn't a
> package so you don't get dependencies handled when you install it? You
> are correct, it's easy to remove the dependencies, I just didn't know it
> was causing problems for people. You can get percentiles from just
> sorting the sample values and indexing into the array at the appropriate
> offset, I was just trying to re-use existing classes.
It's not necessarily a problem, but the less dependencies you have, the
easier it is for people to use. I do the same for fio, try to have as
few external dependencies as possible. Remember, not everybody is
running on Linux...
--
Jens Axboe
next prev parent reply other threads:[~2016-05-24 15:28 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-05-24 10:35 fiologparser.py Martin Steigerwald
2016-05-24 11:12 ` fiologparser.py Ben England
2016-05-24 14:04 ` fiologparser.py Mark Nelson
2016-05-24 14:11 ` fiologparser.py Jens Axboe
2016-05-24 14:17 ` fiologparser.py Mark Nelson
2016-05-24 15:22 ` fiologparser.py Ben England
2016-05-24 15:28 ` Jens Axboe [this message]
2016-05-24 15:35 ` fiologparser.py Ben England
2016-05-24 16:20 ` fiologparser.py Mark Nelson
2016-05-24 18:38 ` fiologparser.py Jeff Furlong
2016-05-24 20:47 ` fiologparser.py Ben England
2016-05-25 2:04 ` fiologparser.py Mark Nelson
2016-05-25 8:58 ` fiologparser.py Martin Steigerwald
2016-05-25 7:20 ` fiologparser.py Martin Steigerwald
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=57447327.3010503@kernel.dk \
--to=axboe@kernel.dk \
--cc=bengland@redhat.com \
--cc=fio@vger.kernel.org \
--cc=mark.a.nelson@gmail.com \
--cc=mnelson@redhat.com \
--cc=ms@teamix.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.