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

creation of extension field of finite field needs more parameter checking #12007

Closed
sagetrac-mariah mannequin opened this issue Nov 10, 2011 · 3 comments
Closed

creation of extension field of finite field needs more parameter checking #12007

sagetrac-mariah mannequin opened this issue Nov 10, 2011 · 3 comments
Assignees
@aghitza
Labels
c: basic arithmetic r: duplicate t: bug

Comments

@sagetrac-mariah
Copy link
Mannequin

sagetrac-mariah mannequin commented Nov 10, 2011

In sage-4.7.2,

sage: F = GF(2)
sage: P.<x> = PolynomialRing(F)
sage: H.<h> = GF(2^8, name='h', modulus=x^3+x+1) # 2^8 rather than 2^3
sage:
sage: len(set([h^i for i in xrange(10000)]))
21

H is something - but it is not a field!

Fixed by #11784.

Component: basic arithmetic

Reviewer: David Roe

Issue created by migration from https://trac.sagemath.org/ticket/12007
@sagetrac-mariah sagetrac-mariah mannequin added this to the sage-5.1 milestone Nov 10, 2011
@sagetrac-mariah sagetrac-mariah mannequin assigned aghitza Nov 10, 2011
@lftabera
Copy link
Contributor

comment:1

This is a duplicate of #11784 that needs review :)

@roed314 roed314 removed this from the sage-5.1 milestone May 25, 2012
@jdemeyer

This comment has been minimized.

Sign in to view
@jdemeyer
Copy link

jdemeyer commented Jun 2, 2012

Reviewer: David Roe

@jdemeyer jdemeyer closed this as completed Jun 2, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@aghitza aghitza

Labels
c: basic arithmetic r: duplicate t: bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@roed314 @aghitza @jdemeyer @lftabera