From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Subject: Re: fiologparser.py References: <12634496.Be1YEipj88@merkaba> <2097283958.65342663.1464088355308.JavaMail.zimbra@redhat.com> <196b983d-e6e1-2c2d-299e-de3b9cc742ff@gmail.com> <1549707390.65536830.1464103326492.JavaMail.zimbra@redhat.com> From: Jens Axboe Message-ID: <57447327.3010503@kernel.dk> Date: Tue, 24 May 2016 09:28:39 -0600 MIME-Version: 1.0 In-Reply-To: <1549707390.65536830.1464103326492.JavaMail.zimbra@redhat.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit To: Ben England , Mark Nelson Cc: Martin Steigerwald , fio@vger.kernel.org, Mark Nelson List-ID: On 05/24/2016 09:22 AM, Ben England wrote: > > > ----- Original Message ----- >> From: "Mark Nelson" >> To: "Ben England" , "Martin Steigerwald" >> Cc: fio@vger.kernel.org, "Mark Nelson" , "Jens Axboe" >> 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