#9880: Segfault in PyNaC 0.2.0.p4
-------------------------+--------------------------------------------------
Reporter: jpflori | Owner: burcin
Type: defect | Status: needs_work
Priority: major | Milestone: sage-4.7
Component: symbolics | Keywords: pynac
Work_issues: | Upstream: N/A
Reviewer: | Author:
Merged: | Dependencies:
-------------------------+--------------------------------------------------
Comment(by jpflori):
Replying to [comment:36 burcin]:
> * doesn't this require a patch to the Sage library at least to fix
doctests? Is the new printing order exactly the same as the old
(inconsistent) one?
I don't think this order and the old one will coincide and a lot of
doctests will have to be fixed if we use it.
I'll try to have a look at all of this soon, I must admit I did not touch
that code since my last post here.
> * Don't we also need to modify the operand access functions (at least
the one in Sage - not the .op() function of ginac) to return the operands
in the sorted order, not the stored (somewhat random) order?
I think you are right.
I did not test it but it should return unexpected values with the new
code.
I'll include that when the new order seems consistent.
> BTW, Jean-Pierre, I wouldn't mind at all if you want to cut the new
pynac release yourself. I can provide instructions on how to do this.
Please go ahead, I'll find the time to do it.
I'm currently rebuilding everything and running some tests, but IIRC the
piece of code Volker provided did not raise errors anymore.
Of course there could be other inconsistencies here and there.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9880#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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
