#5685: [with patch, positive review] enhanced nth_root in ZZ and QQ and related
utilities
------------------------------+---------------------------------------------
Reporter: cremona | Owner: somebody
Type: enhancement | Status: new
Priority: major | Milestone: sage-3.4.2
Component: basic arithmetic | Keywords: nth root rational integer
------------------------------+---------------------------------------------
Comment(by mabshoff):
Hmm, I am not sure what happened, but only the latest rebased patch has
some rejection issues in integer.pyx:
{{{
sage-3.4.1.rc3/devel/sage$ patch -p1 --dry-run < trac_5685_new.patch
patching file sage/rings/integer.pyx
patching file sage/rings/qqbar.py
patching file sage/rings/rational.pyx
patching file sage/schemes/elliptic_curves/ell_rational_field.py
patching file sage/rings/integer.pyx
Hunk #2 FAILED at 1538.
Hunk #3 succeeded at 1564 (offset -13 lines).
Hunk #4 FAILED at 1604.
2 out of 4 hunks FAILED -- saving rejects to file
sage/rings/integer.pyx.rej
patching file sage/rings/number_field/order.py
patching file sage/rings/rational.pyx
Hunk #1 succeeded at 1218 (offset -37 lines).
Hunk #2 succeeded at 1242 (offset -37 lines).
}}}
I will poke around later and see what the problem is.
Cheers,
Michael
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5685#comment:11>
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
-~----------~----~----~----~------~----~------~--~---