Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Disallow is_prime() for FieldElement #17919

Open
jdemeyer opened this issue Mar 9, 2015 · 5 comments
Open

Disallow is_prime() for FieldElement #17919

jdemeyer opened this issue Mar 9, 2015 · 5 comments
Labels
c: basic arithmetic p: minor / 4 t: enhancement

Comments

@jdemeyer
Copy link

jdemeyer commented Mar 9, 2015

The attached branch raises an error when calling is_prime() on a FieldElement.

Component: basic arithmetic

Author: Jeroen Demeyer

Branch/Commit: u/jdemeyer/disallow_is_prime___for_fieldelement @ f0e8c96

Issue created by migration from https://trac.sagemath.org/ticket/17919
@jdemeyer jdemeyer added this to the sage-6.6 milestone Mar 9, 2015
@jdemeyer
Copy link
Author

jdemeyer commented Mar 9, 2015

Branch: u/jdemeyer/disallow_is_prime___for_fieldelement

@jdemeyer
Copy link
Author

jdemeyer commented Mar 9, 2015

New commits:

f0e8c96Raise an error in FieldElement.is_prime()

@jdemeyer
Copy link
Author

jdemeyer commented Mar 9, 2015

Commit: f0e8c96

@jdemeyer

This comment has been minimized.

Sign in to view
@jdemeyer
Copy link
Author

jdemeyer commented Mar 9, 2015

comment:3

Unfortunately, OrderElements are also FieldElement instances, even though the parent is not a field. So this gives doctest failures...

@mkoeppe mkoeppe removed this from the sage-6.6 milestone Dec 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: basic arithmetic p: minor / 4 t: enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jdemeyer @mkoeppe