[enh] fix some pep8 formating errors

evanshultz · Aug 23, 2016 · 6261517 · 6261517
1 parent cd0a103
commit 6261517
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 16 deletions.
diff --git a/pcb/check_kicad_mod.py b/pcb/check_kicad_mod.py
@@ -5,13 +5,13 @@
 import argparse
 from kicad_mod import *
 import sys, os
-#point to the correct location for the print_color script
-sys.path.append(os.path.join(sys.path[0],'..','schlib'))
+# point to the correct location for the print_color script
+sys.path.append(os.path.join(sys.path[0], '..', 'schlib'))
 
 from print_color import *
 from rules import *
 
-#enable windows wildcards
+# enable windows wildcards
 from glob import glob
 
 parser = argparse.ArgumentParser()
@@ -21,7 +21,7 @@
 parser.add_argument('-v', '--verbose', help='show status of all modules and extra information about the violation', action='store_true')
 args = parser.parse_args()
 
-printer = PrintColor(use_color = not args.nocolor)
+printer = PrintColor(use_color=not args.nocolor)
 
 return_val = 0
 
@@ -35,7 +35,7 @@
 
 for f in args.kicad_mod_files:
     files += glob(f)
-        
+
 for filename in files:
     try:
         module = KicadMod(filename)
@@ -51,7 +51,7 @@
         if rule.check():
             n_violations += 1
             return_val = 1
-            printer.yellow('Violating ' +  rule.name, indentation=2)
+            printer.yellow('Violating ' + rule.name, indentation=2)
             if args.verbose:
                 printer.light_blue(rule.description, indentation=4, max_width=100)
 

diff --git a/sch/add_part_number.py b/sch/add_part_number.py
@@ -38,23 +38,23 @@
 
     for component in sch.components:
         # check if is power related component
-        if ('#PWR' in component.fields[0]['ref'] or
-            'PWR_FLAG' in component.fields[1]['ref']):
+        if '#PWR' in component.fields[0]['ref'] or\
+           'PWR_FLAG' in component.fields[1]['ref']:
             continue
 
         # create or get the PN field
         for field in component.fields:
-            if field['name'].replace('"','') == args.pn_field_name:
+            if field['name'].replace('"', '') == args.pn_field_name:
                 break
         else:
-            field = component.addField({'name':'"%s"' % args.pn_field_name, 'ref':'"~"'})
+            field = component.addField({'name': '"%s"' % args.pn_field_name, 'ref': '"~"'})
 
         # component reference
-        comp_ref = component.fields[0]['ref'].replace('"','')
+        comp_ref = component.fields[0]['ref'].replace('"', '')
 
         # search the component in the BOM items and get the PN
         for item in bom:
-            item_refs = item[ref_col].replace(' ','').split(',')
+            item_refs = item[ref_col].replace(' ', '').split(',')
             if item[pn_col] and comp_ref in item_refs:
                 field['ref'] = '"' + item[pn_col] + '"'
                 break

diff --git a/sch/update_footprints.py b/sch/update_footprints.py
@@ -37,16 +37,16 @@
 
     for component in sch.components:
         # check if is power related component
-        if ('#PWR' in component.fields[0]['ref'] or
-            'PWR_FLAG' in component.fields[1]['ref']):
+        if '#PWR' in component.fields[0]['ref'] or\
+           'PWR_FLAG' in component.fields[1]['ref']:
             continue
 
         # component reference
-        comp_ref = component.fields[0]['ref'].replace('"','')
+        comp_ref = component.fields[0]['ref'].replace('"', '')
 
         # search the component in the BOM items and get the PN
         for item in bom:
-            item_refs = item[ref_col].replace(' ','').split(',')
+            item_refs = item[ref_col].replace(' ', '').split(',')
             if item[fp_col] and comp_ref in item_refs:
                 # set component footprint
                 component.fields[2]['ref'] = '"' + item[fp_col] + '"'