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
> >
> >
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/