[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [gpsd-dev] [PATCH] Speed up gpsfake.
From: |
Gary E. Miller |
Subject: |
Re: [gpsd-dev] [PATCH] Speed up gpsfake. |
Date: |
Thu, 17 May 2012 14:05:36 -0700 |
Yo Eric!
i was going to make your suggested change. But found that the regessions
are not working for me at all.
Processing test/daemon/ait250.log...gpsd:ERROR: device open failed: Permission
denied - retrying read-only
gpsd:ERROR: read-only device open failed: Permission denied
gpsd:ERROR: /dev/pts/3: device activation failed.
gpsd:ERROR: device open failed: Permission denied - retrying read-only
gpsd:ERROR: read-only device open failed: Permission denied
gpsd:ERROR: /dev/pts/3: device activation failed.
Any ideas?
On Thu, 17 May 2012 16:31:44 -0400
"Eric S. Raymond" <address@hidden> wrote:
> Beat Bolli <address@hidden>:
> > Hi Terje
> >
> > On 17.05.12 20:35, Terje Mathisen wrote:
> > > The "count % 11" operation is _very_ expensive you know,
>
> It's *Python*. A test tool. Micro-optimizing at this level makes no
> sense - I prefer the clarity of the % operation. Maintainability,
> always think maintainability first. Machine-efficiency gains are
> almost[*] never interesting; they get swamped by Moore's Law too
> quickly.
>
> * Except in well-defined critical paths.
>
> > At least I thought of using a reduction factor that has no common
> > divisors with the number of baton states...
>
> *That* property should have been documented.
>
> # By setting this > 1 we reduce the frequency of the twirl
> # and speed up test runs. Should be relatively prime to the
> # nunber of baton states, otherwise it will cause beat artifacts
> # in the twirling.
> SPINNER_INTERVAL = 11
>
> /me hopes both Terje and Beat will take a lesson here.
RGDS
GARY
---------------------------------------------------------------------------
Gary E. Miller Rellim 109 NW Wilmington Ave., Suite E, Bend, OR 97701
address@hidden Tel:+1(541)382-8588
signature.asc
Description: PGP signature