Trac #31654: docstring improvements

Author: mjungmath

src/sage/manifolds/differentiable/tensorfield.py

@@ -2056,12 +2056,11 @@ def __setitem__(self, args, value):
 
     def copy_from(self, other):
         r"""
-        Make ``self`` to a copy of ``other``.
+        Make ``self`` a copy of ``other``.
 
         INPUT:
 
-        - ``other`` -- other tensor field in the very same module from which
-          ``self`` should be a copy of
+        - ``other`` -- other tensor field, in the same module as ``self``
 
         .. NOTE::
 
@@ -2107,8 +2106,8 @@ def copy_from(self, other):
             raise ValueError("the components of an immutable element "
                              "cannot be changed")
         if other not in self.parent():
-            raise TypeError("the original must be an element "
-                            + "of {}".format(self.parent()))
+            raise TypeError("the original must be an element of "
+                            f"{self.parent()}")
         self._del_derived()
         self._del_restrictions() # delete restrictions
         name, latex_name = self._name, self._latex_name # keep names

src/sage/manifolds/scalarfield.py

@@ -1616,12 +1616,11 @@ def copy(self, name=None, latex_name=None):
 
     def copy_from(self, other):
         r"""
-        Make ``self`` to a copy of ``other``.
+        Make ``self`` a copy of ``other``.
 
         INPUT:
 
-        - ``other`` -- other scalar field in the very same module from which
-          ``self`` should be a copy of
+        - ``other`` -- other scalar field, in the same module as ``self``
 
         .. NOTE::
 
@@ -1664,8 +1663,8 @@ def copy_from(self, other):
             raise ValueError("the expressions of an immutable element "
                              "cannot be changed")
         if other not in self.parent():
-            raise TypeError("the original must be an element "
-                            + "of {}".format(self.parent()))
+            raise TypeError("the original must be an element of "
+                            f"{self.parent()}")
         self._del_derived()
         for chart, funct in other._express.items():
             self._express[chart] = funct.copy()

src/sage/manifolds/section.py

@@ -1646,12 +1646,11 @@ def __setitem__(self, args, value):
 
     def copy_from(self, other):
         r"""
-        Make ``self`` to a copy from ``other``.
+        Make ``self`` a copy of ``other``.
 
         INPUT:
 
-        - ``other`` -- other section in the very same module from which
-          ``self`` should be a copy of
+        - ``other`` -- other section, in the same module as ``self``
 
         .. NOTE::
 
@@ -1702,8 +1701,8 @@ def copy_from(self, other):
             raise ValueError("the components of an immutable element "
                              "cannot be changed")
         if other not in self.parent():
-            raise TypeError("the original must be an element "
-                            + "of {}".format(self.parent()))
+            raise TypeError("the original must be an element of "
+                            f"{self.parent()}")
         self._del_derived()
         self._del_restrictions() # delete restrictions
         name, latex_name = self._name, self._latex_name # keep names