#10836: primitive root is broken
--------------------------------------------------+-------------------------
Reporter: kcrisman | Owner: was
Type: defect | Status: needs_review
Priority: critical | Milestone: sage-4.7
Component: number theory | Keywords:
Author: Karl-Dieter Crisman, William Stein | Upstream: Reported
upstream. Developers deny it's a bug.
Reviewer: William Stein, Karl-Dieter Crisman | Merged:
Work_issues: |
--------------------------------------------------+-------------------------
Changes (by kcrisman):
* status: needs_info => needs_review
Comment:
> Should we throw a better error message than
> {{{
> AttributeError: 'sage.rings.rational.Rational' object has no attribute
'is_prime_power'
> }}}
> for non-integer input?
Actually, we end up with
{{{
TypeError: Attempt to coerce non-integral RealNumber to Integer
}}}
which I didn't notice but is obvious because of the `ZZ(n)`. So that is
okay.
Assuming the docs look ok, I now think that adding the other thing is not
worth the effort. I think I may need to give a reviewer patch to fix a
couple REsT things, though. We'll see :)
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10836#comment:17>
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.
