#4000: [with patch, needs work] Implement QQ['x'] via Flint ZZ['x'] +
denominator
------------------------------+---------------------------------------------
Reporter: malb | Owner: somebody
Type: enhancement | Status: new
Priority: major | Milestone: sage-wishlist
Component: basic arithmetic | Keywords:
Reviewer: | Author: Martin Albrecht
Merged: |
------------------------------+---------------------------------------------
Comment(by malb):
'''Some remarks'''
* the patch uses the old style docstring format, cf.
http://wiki.sagemath.org/combinat/HelpOnTheDoc
* you should claim copyright
* {{{cdef inline int _celement_canonicalise}}} why not {{{void}}}?
* '''all''' {{{celement_foo}}} functions should have doctests, which call
the {{{QQ[x]}}} methods which call the {{{celement_}}} functions
* have you tried switching the default to this implementation to see how
many doctests fail?
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4000#comment:4>
Sage <http://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
-~----------~----~----~----~------~----~------~--~---
