Skip to content

Commit

Permalink
sagemathgh-36078: sage.{interacts,misc,repl,typeset}: Update `# nee…
Browse files Browse the repository at this point in the history
…ds`, modularization fixes

    
<!-- ^^^^^
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 sagemath#1234" use "Introduce new method to
calculate 1+1"
-->
<!-- Describe your changes here in detail -->

<!-- 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 sagemath#12345". -->
- Part of: sagemath#29705
- Cherry-picked from: sagemath#35095
<!-- 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.
- [x] 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
- sagemath#12345: short description why this is a dependency
- sagemath#34567: ...
-->

<!-- If you're unsure about any of these, don't hesitate to ask. We're
here to help! -->
    
URL: sagemath#36078
Reported by: Matthias Köppe
Reviewer(s): David Coudert, Matthias Köppe
  • Loading branch information
Release Manager committed Aug 23, 2023
2 parents 6038123 + bd0fdd5 commit 15fe221
Show file tree
Hide file tree
Showing 78 changed files with 1,384 additions and 1,136 deletions.
4 changes: 2 additions & 2 deletions src/sage/groups/abelian_gps/dual_abelian_group_element.py
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ def is_DualAbelianGroupElement(x) -> bool:
EXAMPLES::
sage: from sage.groups.abelian_gps.dual_abelian_group import is_DualAbelianGroupElement
sage: F = AbelianGroup(5,[5,5,7,8,9],names = list("abcde")).dual_group()
sage: F = AbelianGroup(5, [5,5,7,8,9], names=list("abcde")).dual_group()
sage: is_DualAbelianGroupElement(F)
False
sage: is_DualAbelianGroupElement(F.an_element())
Expand Down Expand Up @@ -146,7 +146,7 @@ def word_problem(self, words):
EXAMPLES::
sage: G = AbelianGroup(5,[3, 5, 5, 7, 8],names="abcde")
sage: G = AbelianGroup(5,[3, 5, 5, 7, 8], names="abcde")
sage: Gd = G.dual_group(names="abcde")
sage: a,b,c,d,e = Gd.gens()
sage: u = a^3*b*c*d^2*e^5
Expand Down
2 changes: 2 additions & 0 deletions src/sage/interacts/library_cython.pyx
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ cpdef julia(ff_j, z, int iterations):
TESTS::
sage: # needs sage.symbolic
sage: from sage.interacts.library_cython import julia
sage: z = var('z')
sage: c_real, c_imag = 1, 1
Expand Down Expand Up @@ -55,6 +56,7 @@ cpdef mandel(ff_m, z, int iterations):
TESTS::
sage: # needs sage.symbolic
sage: from sage.interacts.library_cython import mandel
sage: z, c = var('z, c')
sage: f = symbolic_expression(z**2 + c).function(z,c)
Expand Down
6 changes: 3 additions & 3 deletions src/sage/misc/binary_tree.pyx
Original file line number Diff line number Diff line change
Expand Up @@ -285,9 +285,9 @@ cdef class BinaryTree:
sage: from sage.misc.binary_tree import BinaryTree
sage: t = BinaryTree()
sage: t.insert(0,Matrix([[0,0],[1,1]]))
sage: t.insert(0,1)
sage: t.get(0)
sage: t.insert(0, Matrix([[0,0], [1,1]])) # needs sage.modules
sage: t.insert(0, 1)
sage: t.get(0) # needs sage.modules
[0 0]
[1 1]
"""
Expand Down
8 changes: 4 additions & 4 deletions src/sage/misc/bindable_class.py
Original file line number Diff line number Diff line change
Expand Up @@ -175,22 +175,22 @@ class BoundClass(functools.partial):
.. warning::
Since ``c`` is not a class (as tested by inspect.isclass),
Since ``c`` is not a class (as tested by :func:`inspect.isclass`),
and has a ``__call__`` method, IPython's introspection
(with ``c?``) insists on showing not only its
documentation but also its class documentation and call
documentation (see :meth:`IPython.OInspect.Inspector.pdoc`)
if available.
Until a better approach is found, we reset the documentation
of ``BoundClass`` below, and make an exception for
of :class:`BoundClass` below, and make an exception for
:meth:`__init__` to the strict rule that every method should
be doctested::
sage: c.__class__.__doc__
sage: c.__class__.__init__.__doc__
Make sure classes which inherit from functools.partial have the correct
Make sure classes which inherit from :class:`functools.partial` have the correct
syntax, see :trac:`14748`::
sage: import warnings
Expand All @@ -201,7 +201,7 @@ class BoundClass(functools.partial):
sage: g()
8
The following has correct syntax and no ``DeprecationWarning``::
The following has correct syntax and no :class:`DeprecationWarning`::
sage: class mynewpartial(functools.partial):
....: def __init__(self, f, i, j):
Expand Down
Loading

0 comments on commit 15fe221

Please sign in to comment.