Re: [sage-devel] 0 should not be a primitive root

2017-08-16 Thread Will Song
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 > > > >

Re: [sage-devel] 0 should not be a primitive root

2017-08-16 Thread Daniel Krenn
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/