Skip to content

Commit f71381c

Browse files
author
Release Manager
committed
gh-37279: Fix some doctest warnings <!-- ^^^^^ Please provide a concise, informative and self-explanatory title. Don't put issue numbers in there, do this in the PR body below. For example, instead of "Fixes #1234" use "Introduce new method to calculate 1+1" --> <!-- Describe your changes here in detail --> These are some of the last remaining doctest warnings as of 10.3.beta7, as seen e.g. in https://github.com/sagemath/sage/pull/37250/files "Unchanged files with check annotations". The ones not fixed here will need a solution for: - #36099 <!-- Why is this change required? What problem does it solve? --> <!-- If this PR resolves an open issue, please link to it here. For example "Fixes #12345". --> <!-- If your change requires a documentation PR, please link it appropriately. --> ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> <!-- If your change requires a documentation PR, please link it appropriately --> <!-- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> <!-- Feel free to remove irrelevant items. --> - [x] The title is concise, informative, and self-explanatory. - [ ] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation accordingly. ### ⌛ Dependencies <!-- List all open PRs that this PR logically depends on - #12345: short description why this is a dependency - #34567: ... --> <!-- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> URL: #37279 Reported by: Matthias Köppe Reviewer(s): David Coudert
2 parents 36bf4af + 9bfc29a commit f71381c

File tree

5 files changed

+11
-10
lines changed

5 files changed

+11
-10
lines changed

src/sage/geometry/polyhedron/backend_field.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ def _is_zero(self, x):
9090
sage: p = Polyhedron([(sqrt(3),sqrt(2))], base_ring=AA) # needs sage.rings.number_field sage.symbolic
9191
sage: p._is_zero(0) # needs sage.rings.number_field sage.symbolic
9292
True
93-
sage: p._is_zero(1/100000) # needs sage.rings.number_field
93+
sage: p._is_zero(1/100000) # needs sage.rings.number_field sage.symbolic
9494
False
9595
"""
9696
return x == 0

src/sage/rings/finite_rings/integer_mod_ring.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -617,14 +617,14 @@ def multiplicative_subgroups(self):
617617
618618
EXAMPLES::
619619
620-
sage: # needs sage.groups
621-
sage: Integers(5).multiplicative_subgroups() # optional - gap_package_polycyclic
620+
sage: # optional - gap_package_polycyclic, needs sage.groups
621+
sage: Integers(5).multiplicative_subgroups()
622622
((2,), (4,), ())
623-
sage: Integers(15).multiplicative_subgroups() # optional - gap_package_polycyclic
623+
sage: Integers(15).multiplicative_subgroups()
624624
((11, 7), (11, 4), (2,), (11,), (14,), (7,), (4,), ())
625-
sage: Integers(2).multiplicative_subgroups() # optional - gap_package_polycyclic
625+
sage: Integers(2).multiplicative_subgroups()
626626
((),)
627-
sage: len(Integers(341).multiplicative_subgroups()) # optional - gap_package_polycyclic
627+
sage: len(Integers(341).multiplicative_subgroups())
628628
80
629629
630630
TESTS::
@@ -633,7 +633,7 @@ def multiplicative_subgroups(self):
633633
((),)
634634
sage: IntegerModRing(2).multiplicative_subgroups() # needs sage.groups
635635
((),)
636-
sage: IntegerModRing(3).multiplicative_subgroups() # needs sage.groups # optional - gap_package_polycyclic
636+
sage: IntegerModRing(3).multiplicative_subgroups() # optional - gap_package_polycyclic, needs sage.groups
637637
((2,), ())
638638
"""
639639
return tuple(tuple(g.value() for g in H.gens())

src/sage/rings/generic.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ class ProductTree:
3636
Similarly, the :meth:`interpolation` method can be used to implement
3737
the inverse Fast Fourier Transform::
3838
39-
sage: tree.interpolation(zs).padded_list(len(ys)) == ys
39+
sage: tree.interpolation(zs).padded_list(len(ys)) == ys # needs sage.rings.finite_rings
4040
True
4141
4242
This class encodes the tree as *layers*: Layer `0` is just a tuple

src/sage/rings/polynomial/polynomial_rational_flint.pyx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2127,11 +2127,11 @@ cdef class Polynomial_rational_flint(Polynomial):
21272127
21282128
::
21292129
2130-
sage: # needs sage.libs.pari
2130+
sage: # needs sage.groups sage.libs.pari
21312131
sage: f = x^4 - 17*x^3 - 2*x + 1
21322132
sage: G = f.galois_group(pari_group=True); G
21332133
PARI group [24, -1, 5, "S4"] of degree 4
2134-
sage: PermutationGroup(G) # needs sage.groups
2134+
sage: PermutationGroup(G)
21352135
Transitive group number 5 of degree 4
21362136
21372137
You can use KASH or GAP to compute Galois groups as well. The advantage is

src/sage/schemes/curves/affine_curve.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1801,6 +1801,7 @@ def fundamental_group(self, simplified=True, puiseux=False):
18011801
to the algebraic field::
18021802
18031803
sage: # needs sage.rings.number_field
1804+
sage: x = polygen(ZZ)
18041805
sage: a = QQ[x](x^2 + 5).roots(QQbar)[0][0]
18051806
sage: F = NumberField(a.minpoly(), 'a', embedding=a)
18061807
sage: F.inject_variables()

0 commit comments

Comments
 (0)