Re: fiologparser.py

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



OK we'll remove the dependencies, I still want to have the -A option supported.
-ben

----- Original Message -----
> From: "Jens Axboe" <axboe@xxxxxxxxx>
> To: "Ben England" <bengland@xxxxxxxxxx>, "Mark Nelson" <mark.a.nelson@xxxxxxxxx>
> Cc: "Martin Steigerwald" <ms@xxxxxxxxx>, fio@xxxxxxxxxxxxxxx, "Mark Nelson" <mnelson@xxxxxxxxxx>
> Sent: Tuesday, May 24, 2016 11:28:39 AM
> Subject: Re: fiologparser.py
> 
> On 05/24/2016 09:22 AM, Ben England wrote:
> >
> >
> > ----- Original Message -----
> >> From: "Mark Nelson" <mark.a.nelson@xxxxxxxxx>
> >> To: "Ben England" <bengland@xxxxxxxxxx>, "Martin Steigerwald"
> >> <ms@xxxxxxxxx>
> >> Cc: fio@xxxxxxxxxxxxxxx, "Mark Nelson" <mnelson@xxxxxxxxxx>, "Jens Axboe"
> >> <axboe@xxxxxxxxx>
> >> 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
> 
> 
--
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



[Index of Archives]     [Linux Kernel]     [Linux SCSI]     [Linux IDE]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux SCSI]

  Powered by Linux