Opened 9 years ago

Closed 9 years ago

#9610 closed defect (fixed)

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

Reported by: rlm Owned by: was
Priority: major Milestone: sage-4.5.3
Component: number theory Keywords:
Cc: malb Merged in: sage-4.5.3.alpha0
Authors: Robert Miller Reviewers: Martin Albrecht
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

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

Attachments (1)

trac_9610.patch (7.3 KB) - added by rlm 9 years ago.

Download all attachments as: .zip

Change History (4)

Changed 9 years ago by rlm

comment:1 Changed 9 years ago by malb

  • Reviewers set to Martin Albrecht
  • Status changed from new to needs_review

Looks good.

comment:2 Changed 9 years ago by malb

  • Status changed from needs_review to positive_review

comment:3 Changed 9 years ago by mpatel

  • Merged in set to sage-4.5.3.alpha0
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.