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.

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

int(finite field element) should only work when it is in the prime subfield #9610

Closed
rlmill mannequin opened this issue Jul 27, 2010 · 4 comments
Closed

int(finite field element) should only work when it is in the prime subfield #9610

rlmill mannequin opened this issue Jul 27, 2010 · 4 comments

Comments

@rlmill
Copy link
Mannequin

rlmill mannequin commented Jul 27, 2010

This was the real cause of #8406, and the fix there introduced the following bug:

----------------------------------------------------------------------
| Sage Version 4.5, Release Date: 2010-07-16                         |
| Type notebook() for the GUI, and license() for information.        |
----------------------------------------------------------------------
sage: n = 20
sage: k = 3
sage: g = DiGraph()
sage: g.add_edges( (i,Mod(i+j,n)) for i in range(n) for j in range(1,k+1) )
sage: g.vertices()
[0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19]
sage: g.strongly_connected_components()
[[0], [0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19]]

CC: @malb

Component: number theory

Author: Robert Miller

Reviewer: Martin Albrecht

Merged: sage-4.5.3.alpha0

Issue created by migration from https://trac.sagemath.org/ticket/9610

@rlmill rlmill mannequin added this to the sage-4.5.3 milestone Jul 27, 2010
@rlmill rlmill mannequin added c: number theory labels Jul 27, 2010
@rlmill rlmill mannequin assigned williamstein Jul 27, 2010
@rlmill
Copy link
Mannequin Author

rlmill mannequin commented Jul 27, 2010

Attachment: trac_9610.patch.gz

@malb
Copy link
Member

malb commented Jul 27, 2010

Reviewer: Martin Albrecht

@malb
Copy link
Member

malb commented Jul 27, 2010

comment:1

Looks good.

@qed777
Copy link
Mannequin

qed777 mannequin commented Aug 9, 2010

Merged: sage-4.5.3.alpha0

@qed777 qed777 mannequin removed the s: positive review label Aug 9, 2010
@qed777 qed777 mannequin closed this as completed Aug 9, 2010
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants