#10295: Upgrading pexpect
-------------------------------------+-------------------------------------
Reporter: SimonKing | Owner: was
Type: enhancement | Status: new
Priority: major | Milestone: sage-6.7
Component: interfaces | Resolution:
Keywords: pexpect upgrade | Merged in:
Authors: | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/fbissey/pexpect3.3 | a24eab3ce985874ab8445ed5e362bbeead76fd40
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by bpage):
Apparently the performance problem originates with this old commit:
https://github.com/pexpect/pexpect/commit/600463686284ea102548f5e0bf51582db051dc78
#diff-dfb232b43530f0961894e09a0d862aa4R1048
after commenting out
https://github.com/pexpect/pexpect/blob/3.x/pexpect/__init__.py#L1540
{{{
#time.sleep(0.0001)
}}}
performance changes from
{{{
sage: sage: %timeit gp.eval('2+3')
1000 loops, best of 3: 427 µs per loop
sage: sage: %timeit gp.eval('2+3')
1000 loops, best of 3: 449 µs per loop
}}}
to this
{{{
sage: %timeit gp.eval('2+3')
The slowest run took 175.25 times longer than the fastest. This could mean
that an intermediate result is being cached
10000 loops, best of 3: 76.3 µs per loop
sage: %timeit gp.eval('2+3')
The slowest run took 7.85 times longer than the fastest. This could mean
that an intermediate result is being cached
10000 loops, best of 3: 76.4 µs per loop
}}}
--
Ticket URL: <http://trac.sagemath.org/ticket/10295#comment:37>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
