Thanks for catching my mistake it should indeed be False instead of True.

On Wednesday, August 16, 2017 at 3:00:41 AM UTC-6, Daniel Krenn wrote:
>
> On 2017-08-14 18:37, Will Song wrote: 
> > Not sure if this is a known bug or not. 
> > 
> > SageMath version 8.0, Release Date: 2017-07-21 
> > 
> > sage: mod(0, 17).is_primitive_root() 
> > True 
> > 
> > The easy fix would just be to add 
> > 
> > if self == 0: 
> >     return True 
> > 
> > after 
> https://github.com/sagemath/sage/blob/master/src/sage/rings/finite_rings/integer_mod.pyx#L1586.
>  
>
> > This also fixes the issue with p^j being a primitive root mod p^k where 
> > p is an odd prime. 
>
> This is now 
> https://trac.sagemath.org/ticket/23624 
>
>

-- 
You received this message because you are subscribed to the Google Groups 
"sage-devel" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to sage-devel+unsubscr...@googlegroups.com.
To post to this group, send email to sage-devel@googlegroups.com.
Visit this group at https://groups.google.com/group/sage-devel.
For more options, visit https://groups.google.com/d/optout.

Reply via email to