#4274: [with extra patch; needs review (quick)] assertion failure in rank for
elliptic curves
--------------------------------+-------------------------------------------
Reporter: zimmerma | Owner: was
Type: defect | Status: new
Priority: minor | Milestone: sage-3.4.1
Component: algebraic geometry | Resolution:
Keywords: |
--------------------------------+-------------------------------------------
Changes (by cremona):
* summary: [with patch; with positive review but one oddity] assertion
failure in rank for elliptic curves => [with
extra patch; needs review (quick)] assertion
failure in rank for elliptic curves
Comment:
I have sorted this. the problem was that the RuntimeError output string
included the whole of the mwrank output, but the doctesting machinery soes
not seem to be able to strip bits out of that string, only normal output.
So I changed it so that (in the case in question), first the mwrank output
string is output, then the RuntimeError is raised with a short message.
At the same time i trimmed the long mwrank output so that only the short
extract from near the end is output. People who want to see the whole lot
can always call mwrank() on the curve.
Additional patch does this, and fixes the problem -- minor review only
required, I think!
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4274#comment:8>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---
