Commit c76e573
Release Manager
sagemathgh-40881: sage.categories.finite_fields: don't import from integer_ring
Importing `ZZ` from the integer ring creates an import cycle,
```
integer_ring -> integer
-> rational
-> rational_field
-> categories/finite_fields
-> integer_ring
```
I noticed this because the cycle has started causing an `ImportError`
when I attempt to import `ZZ`. This commit doesn't fix the problem
(there are other cycles), but it is minimally invasive and does improve
the import graph a tiny bit.
Edit: a better fix for this would be to break the dependence of integers
on rationals, but that would be a much larger change.
URL: sagemath#40881
Reported by: Michael Orlitzky
Reviewer(s): Dima Pasechnik
1 file changed
+1
-2
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
16 | 16 | | |
17 | 17 | | |
18 | 18 | | |
19 | | - | |
20 | 19 | | |
21 | 20 | | |
22 | 21 | | |
| |||
332 | 331 | | |
333 | 332 | | |
334 | 333 | | |
335 | | - | |
| 334 | + | |
336 | 335 | | |
337 | 336 | | |
338 | 337 | | |
| |||
0 commit comments