#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 leif):
Replying to [comment:47 bpage]:
> The pexpect people might accept a patch to change this to 0.
Well, rather the delay should be configurable.
But the whole `expect_loop()` and `read_nonblocking()` look pretty messy.
We should perhaps simply overwrite one or both with a version that fits
our needs. The whole class certainly isn't written for performance.
Note that the behaviour also depends on the writer (i.e., the subprocess);
ECL for example is extremely poor in that it doesn't write strings, but
really character by character, such that in the worst case we here would
end up with at least `len(s)` * 0.1 ms (and at least len(s) context
switches, not including the system calls themselves, where pexpect doesn't
only call `select()` and probably `read()`, but also `waitpid()` because
of the `isalive()` checks).
--
Ticket URL: <http://trac.sagemath.org/ticket/10295#comment:49>
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.
