a bit more in plot and modules

Author: fchapoton

src/sage/modules/fg_pid/fgp_element.py

@@ -21,7 +21,7 @@
 # This adds extra maybe-not-necessary checks in the code, but could
 # slow things down.  It can impact what happens in more than just this
 # file.
-DEBUG=True
+DEBUG = True
 
 
 class FGP_Element(ModuleElement):

src/sage/modules/fg_pid/fgp_morphism.py

@@ -127,7 +127,7 @@ def _repr_(self):
             sage: phi._repr_()
             'Morphism from module over Integer Ring with invariants (4, 12) to module with invariants (4, 12) that sends the generators to [(1, 3), (0, 11)]'
         """
-        return "Morphism from module over %s with invariants %s to module with invariants %s that sends the generators to %s"%(
+        return "Morphism from module over %s with invariants %s to module with invariants %s that sends the generators to %s" % (
             self.domain().base_ring(), self.domain().invariants(), self.codomain().invariants(),
             list(self.im_gens()))
 

src/sage/modules/with_basis/morphism.py

@@ -691,9 +691,9 @@ def __init__(self, triangular="upper", unitriangular=False,
         self._inverse_on_support = inverse_on_support
 
         if invertible is None and (domain.basis().keys() == codomain.basis().keys()) and \
-           (self._inverse_on_support==identity or domain in Modules.FiniteDimensional):
+           (self._inverse_on_support == identity or domain in Modules.FiniteDimensional):
             invertible = True
-        self._invertible=invertible
+        self._invertible = invertible
 
     def _richcmp_(self, other, op):
         r"""
@@ -772,10 +772,10 @@ def _test_triangular(self, **options):
             bs, co = self._dominant_item(on_basis(x))
             if self._unitriangular:
                 tester.assertEqual(co, self.domain().base_ring().one(),
-                    LazyFormat("morphism is not unitriangular on %s")%(x))
+                    LazyFormat("morphism is not unitriangular on %s") % (x))
             xback = self._inverse_on_support(bs)
             tester.assertEqual(x, xback,
-                LazyFormat("morphism is not triangular on %s")%(x))
+                LazyFormat("morphism is not triangular on %s") % (x))
 
     def __invert__(self):
         """
@@ -1452,12 +1452,12 @@ def __init__(self, domain, diagonal, codomain=None, category=None):
                              "and the same basis indexing")
         from collections.abc import Callable
         if not isinstance(diagonal, Callable):
-            raise ValueError("diagonal (=%s) should be a function"%diagonal)
+            raise ValueError("diagonal (=%s) should be a function" % diagonal)
         if category is None:
             category = ModulesWithBasis(domain.base_ring())
         ModuleMorphismByLinearity.__init__(
             self, domain=domain, codomain=codomain, category=category)
-        self._diagonal=diagonal
+        self._diagonal = diagonal
 
     def _richcmp_(self, other, op):
         r"""

src/sage/plot/plot3d/plot3d.py

@@ -263,7 +263,7 @@ def __init__(self, dep_var, indep_vars):
         """
         all_vars = sage_getargspec(self.transform).args[1:]
         if set(all_vars) != set(indep_vars + [dep_var]):
-            raise ValueError('variables were specified incorrectly for this coordinate system; incorrect variables were %s'%list(set(all_vars).symmetric_difference(set(indep_vars+[dep_var]))))
+            raise ValueError('variables were specified incorrectly for this coordinate system; incorrect variables were %s' % list(set(all_vars).symmetric_difference(set(indep_vars+[dep_var]))))
         self.dep_var = dep_var
         self.indep_vars = indep_vars
 
@@ -1372,20 +1372,20 @@ def plot3d(f, urange, vrange, adaptive=False, transformation=None, **kwds):
 
         from sage.modules.vector_callable_symbolic_dense import Vector_callable_symbolic_dense
         if isinstance(transformation, (tuple, list,Vector_callable_symbolic_dense)):
-            if len(transformation)==3:
+            if len(transformation) == 3:
                 if params is None:
                     raise ValueError("must specify independent variable names in the ranges when using generic transformation")
                 indep_vars = params
-            elif len(transformation)==4:
+            elif len(transformation) == 4:
                 indep_vars = transformation[3]
                 transformation = transformation[0:3]
             else:
                 raise ValueError("unknown transformation type")
             # find out which variable is the function variable by
             # eliminating the parameter variables.
             all_vars = set(sum([list(s.variables()) for s in transformation],[]))
-            dep_var=all_vars - set(indep_vars)
-            if len(dep_var)==1:
+            dep_var = all_vars - set(indep_vars)
+            if len(dep_var) == 1:
                 dep_var = dep_var.pop()
                 transformation = _ArbitraryCoordinates(transformation, dep_var, indep_vars)
             else:

src/sage/plot/plot3d/tri_plot.py

@@ -64,7 +64,7 @@ class of the form
             sage: print(tri.str())
             [0, 0, 0] [-1, 2, 3] [0, 2, 0] 0
         """
-        return "%s %s %s %s"%(self._a, self._b, self._c, self._color)
+        return "%s %s %s %s" % (self._a, self._b, self._c, self._color)
 
     def set_color(self, color):
         """
@@ -135,7 +135,7 @@ def str(self):
             sage: print(t.str())
             [1, 2, 3] [2, 3, 4] [0, 0, 0] 0 [0, 0, 1] [0, 1, 0] [1, 0, 0]
         """
-        return "%s %s %s %s %s %s %s"%(self._a, self._b, self._c, self._color, self._da, self._db, self._dc)
+        return "%s %s %s %s %s %s %s" % (self._a, self._b, self._c, self._color, self._da, self._db, self._dc)
 
     def get_normals(self):
         """