flake8: fix E262 + E265 inline + block comment should start with '# '

printer.py is left untouched to reduce chance of conflicts

Author: eric-wieser

galgebra/dop.py

@@ -58,7 +58,7 @@ def _eval_derivative_n_times_terms(terms, x, n):
         for k, term in enumerate(terms):
             dc = _basic_diff(term[0], x)
             pd = _basic_diff(term[1], x)
-            #print 'D0, term, dc, pd =', D0, term, dc, pd
+            # print 'D0, term, dc, pd =', D0, term, dc, pd
             if dc != 0:
                 new_terms.append((dc, term[1]))
             if pd != 0:

galgebra/ga.py

@@ -477,7 +477,7 @@ def __init__(self, bases, *, wedge=True, **kwargs):
             if self.e_sq == S(0):
                 self.sing_flg = True
                 print('!!!!If I**2 = 0, I cannot be normalized!!!!')
-                #raise ValueError('!!!!If I**2 = 0, I cannot be normalized!!!!')
+                # raise ValueError('!!!!If I**2 = 0, I cannot be normalized!!!!')
             if self.e_sq > S(0):
                 self.i = self.e/sqrt(self.e_sq)
                 self.i_inv = self.i
@@ -951,7 +951,7 @@ def _build_connection(self):
 
     ######## Functions for Calculation products of blades/bases ########
 
-    #******************** Geometric Product (*) ***********************#
+    # ******************* Geometric Product (*) ********************** #
 
     def geometric_product_basis_blades(self, blade12):
         # geometric (*) product for orthogonal basis
@@ -1037,9 +1037,9 @@ def reduce_basis(self, blst):
                         blst_flg[i] = tmp[2]
                     else:  # blst_expand[i] revised
                         blst_coef[i] = -blst_coef[i]
-                        #if revision force one more pass in case revision
-                        #causes repeated index previous to revised pair of
-                        #indexes
+                        # if revision force one more pass in case revision
+                        # causes repeated index previous to revised pair of
+                        # indexes
                         blst_flg[i] = False
                         blst_expand[i] = tmp[3]
                         blst_coef.append(-blst_coef[i] * tmp[0])
@@ -1110,7 +1110,7 @@ def reduce_basis_loop(g, blst):
 
         return True  # revision complete, blst in normal order
 
-    #******************* Outer/wedge (^) product **********************#
+    # ****************** Outer/wedge (^) product ********************* #
 
     @staticmethod
     def blade_reduce(lst):
@@ -1152,7 +1152,7 @@ def wedge_product_basis_blades(self, blade12):  # blade12 = blade1*blade2
         else:
             return S(0)
 
-    #****** Dot (|) product, reft (<) and right (>) contractions ******#
+    # ***** Dot (|) product, reft (<) and right (>) contractions ***** #
 
     def _dot_product_grade(self, grade1, grade2, mode):
         """
@@ -2010,7 +2010,7 @@ def __init__(self, __u, __coords, *, ga, norm=False, name=None, root='e', debug=
             Base Geometric Algebra
         """
 
-        #print '!!!Enter Sm!!!'
+        # print '!!!Enter Sm!!!'
 
         if printer.GaLatexPrinter.latex_flg:
             printer.GaLatexPrinter.restore()
@@ -2033,19 +2033,19 @@ def __init__(self, __u, __coords, *, ga, norm=False, name=None, root='e', debug=
         basis_str = basis_str[:-1]
         """
 
-        #print 'u =', u
+        # print 'u =', u
 
-        if isinstance(u,mv.Mv):  #Define vector manifold
+        if isinstance(u,mv.Mv):  # Define vector manifold
             self.ebasis = []
             for coord in coords:
-                #Partial derivation of vector function to get basis vectors
+                # Partial derivation of vector function to get basis vectors
                 self.ebasis.append(u.diff(coord))
 
-            #print 'sm ebasis =', self.ebasis
+            # print 'sm ebasis =', self.ebasis
 
             self.g = []
             for b1 in self.ebasis:
-                #Metric tensor from dot products of basis vectors
+                # Metric tensor from dot products of basis vectors
                 tmp = []
                 for b2 in self.ebasis:
                     tmp.append(b1 | b2)
@@ -2063,12 +2063,12 @@ def __init__(self, __u, __coords, *, ga, norm=False, name=None, root='e', debug=
                     tmp.append(diff(x_i, u_j))
                 dxdu.append(tmp)
 
-            #print 'dxdu =', dxdu
+            # print 'dxdu =', dxdu
 
             sub_pairs = list(zip(ga.coords, u))
 
-            #Construct metric tensor form coordinate maps
-            g = eye(n_sub)  #Zero n_sub x n_sub sympy matrix
+            # Construct metric tensor form coordinate maps
+            g = eye(n_sub)  # Zero n_sub x n_sub sympy matrix
             n_range = list(range(n_sub))
             for i in n_range:
                 for j in n_range:
@@ -2083,8 +2083,8 @@ def __init__(self, __u, __coords, *, ga, norm=False, name=None, root='e', debug=
 
         Ga.__init__(self, root, g=g, coords=coords, norm=norm, debug=debug)
 
-        if isinstance(u,mv.Mv):  #Construct additional functions for vector manifold
-            #self.r_basis_mv under construction
+        if isinstance(u,mv.Mv):  # Construct additional functions for vector manifold
+            # self.r_basis_mv under construction
 
             pass
 

galgebra/lt.py

@@ -155,7 +155,7 @@ class Lt(object):
 
     @staticmethod
     def setup(ga):
-        #coords = [Symbol('mu_' + str(x)) for x in ga.coords]
+        # coords = [Symbol('mu_' + str(x)) for x in ga.coords]
         coords = ga.coords
         x = sum([coords[i] * ga.basis[i] for i in ga.n_range])
         return coords, x
@@ -203,7 +203,7 @@ def __init__(self, *args, ga, f=False, mode='g'):
                 for (lt_i, base) in zip(mat_rep, self.Ga.basis):
                     self.lt_dict[base] = lt_i
             else:
-                #mat_rep = map(list, zip(*mat_rep))  # Transpose list of lists
+                # mat_rep = map(list, zip(*mat_rep))  # Transpose list of lists
                 for (row, base1) in zip(mat_rep, self.Ga.basis):
                     tmp = 0
                     for (col, base2) in zip(row, self.Ga.basis):

galgebra/mv.py

@@ -132,7 +132,7 @@ def characterise_Mv(self):
                 args = [obj]
 
         grades = []
-        #print 'args =', args
+        # print 'args =', args
         self.is_blade_rep = True
         for term in args:
             if term.is_commutative:
@@ -141,7 +141,7 @@ def characterise_Mv(self):
             else:
                 c, nc = term.args_cnc(split_1=False)
                 blade = nc[0]
-                #print 'blade =',blade
+                # print 'blade =',blade
                 if blade in self.Ga.blades.flat:
                     grade = self.Ga.blades_to_grades_dict[blade]
                     if grade not in grades:
@@ -349,9 +349,9 @@ def __init__(self, *args, ga, recp=None, coords=None, **kwargs):
                 self.is_blade_rep = x.is_blade_rep
                 self.i_grade = x.i_grade
             else:
-                if isinstance(x, Expr):  #copy constructor for obj expression
+                if isinstance(x, Expr):  # copy constructor for obj expression
                     self.obj = x
-                else:  #copy constructor for scalar obj expression
+                else:  # copy constructor for scalar obj expression
                     self.obj = S(x)
                 self.is_blade_rep = True
                 self.characterise_Mv()
@@ -443,7 +443,7 @@ def __hash__(self):
     def __eq__(self, A):
         if isinstance(A, Mv):
             diff = (self - A).expand().simplify()
-            #diff = (self - A).expand()
+            # diff = (self - A).expand()
             if diff.obj == S(0):
                 return True
             else:
@@ -724,7 +724,7 @@ def append_plus(c_str):
         s = ''
         for (index, (coef, base, grade)) in sorted_terms:
             coef = printer.coef_simplify(coef)
-            #coef = simplify(coef)
+            # coef = simplify(coef)
             l_coef = printer.latex(coef)
             if l_coef == '1' and base != S(1):
                 l_coef = ''
@@ -988,7 +988,7 @@ def blade_coefs(self, blade_lst=None):
         if blade_lst is None:
             blade_lst = self.Ga.mv_blades.flat
 
-        #print 'Enter blade_coefs blade_lst =', blade_lst, type(blade_lst), [i.is_blade() for i in blade_lst]
+        # print 'Enter blade_coefs blade_lst =', blade_lst, type(blade_lst), [i.is_blade() for i in blade_lst]
 
         for blade in blade_lst:
             if not blade.is_base() or not blade.is_blade():
@@ -1994,7 +1994,7 @@ def rot(itheta, A, hint='-'):  # Rotate by the 2-blade itheta the multivector A
         raise ValueError('A not a multivector in rotate(A,itheta)')
 
 
-def refl(B, A):  #  Project on the blade B the multivector A
+def refl(B, A):  # Project on the blade B the multivector A
     if isinstance(A,Mv):
         return A.reflect_in_blade(B)
     else:

setup.cfg

@@ -32,22 +32,17 @@ per-file-ignores =
     ./galgebra/deprecated.py:E231,E266
 
     # E231 missing whitespace after ','
-    # E265 block comment should start with '# '
     # E266 too many leading '#' for block comment
-    ./galgebra/dop.py:E231,E265,E266
+    ./galgebra/dop.py:E231,E266
 
     # E231 missing whitespace after ','
     # E231 missing whitespace after ':'
-    # E262 inline comment should start with '# '
-    # E265 block comment should start with '# '
     # E266 too many leading '#' for block comment
-    ./galgebra/ga.py:E231,E231,E262,E265,E266
+    ./galgebra/ga.py:E231,E231,E266
 
     # E231 missing whitespace after ','
-    # E262 inline comment should start with '# '
-    # E265 block comment should start with '# '
     # F401 'sys' imported but unused
-    ./galgebra/lt.py:E231,E262,E265,F401
+    ./galgebra/lt.py:E231,F401
 
     # E124 closing bracket does not match visual indentation
     # E125 continuation line with same indent as next logical line
@@ -57,10 +52,8 @@ per-file-ignores =
     ./galgebra/metric.py:E124,E125,E231,F821,F841
 
     # E231 missing whitespace after ','
-    # E262 inline comment should start with '# '
-    # E265 block comment should start with '# '
     # E266 too many leading '#' for block comment
-    ./galgebra/mv.py:E231,E262,E265,E266
+    ./galgebra/mv.py:E231,E266
 
     # E117 over-indented
     # E122 continuation line missing indentation or outdented