Merge branch 'spinler-fix-python_style' into 'devel'

Fix Python codestyle

See merge request ndk/ndk-fpga!253

Author: martinspinler

build/scripts/multi_ver/multi_ver.py

@@ -7,13 +7,13 @@
 import os
 from os import system
 from importlib.machinery import SourceFileLoader
-from random import randint
 import time
 from multi_ver_utils import reduce_combinations, create_setting_from_combination
 
 
 FAIL = False
 
+
 # Modify package file according to setting
 def apply_setting(pkg_file, setting, sed_str):
     env = {}
@@ -30,8 +30,6 @@ def apply_setting(pkg_file, setting, sed_str):
 
 # Run Modelsim with the current test_pkg file
 def run_modelsim(fdo_file, test_name, manual=False, gui=False, coverage=False, env={}):
-    global FAIL
-
     logfile_name = f"transcript_{test_name}"
 
     command  = f"do {fdo_file};"

build/scripts/multi_ver/multi_ver_cocotb.py

@@ -6,13 +6,13 @@
 import os
 from os import system
 from importlib.machinery import SourceFileLoader
-from random import randint
 import time
 import os.path as op
 from multi_ver_utils import reduce_combinations, create_setting_from_combination
 
 FAIL = False
 
+
 def junit(testResults: str, allowEmptyResults: bool = False) -> bool:
     if not op.exists(testResults):
         if allowEmptyResults:
@@ -41,6 +41,7 @@ def find_venv() -> str | None:
 
     return None
 
+
 def run_modelsim(settings: dict, venv: str | None = None, gui=False):
     command = ""
 
@@ -73,6 +74,7 @@ def run_modelsim(settings: dict, venv: str | None = None, gui=False):
 # Parsing script arguments
 ##########
 
+
 parser = argparse.ArgumentParser()
 
 parser.add_argument("settings_file", help="Name of verification settings \".py\" file containing \"SETTINGS\" dictionary variable")

build/scripts/multi_ver/multi_ver_utils.py

@@ -5,6 +5,7 @@
 
 from random import randint
 
+
 # RANDOMLY reduce number of combination to a certain percentage
 def reduce_combinations(combinations, reduction_perc=100):
     if reduction_perc >= 100:

comp/axis_tools/flow/switch/top/python/common/hdr_info.py

@@ -15,8 +15,8 @@
     'name'  : Ether._name,
     'width' : 14,
     'fields': {
-        ( 47,  0): 'dst',
-        ( 95, 48): 'src',
+        (47,  0): 'dst',
+        (95, 48): 'src',
         (111, 96): 'type',
     }
 }
@@ -51,18 +51,22 @@
     2: HDR_VLAN_1AD,
 }
 
-def get_protocol_class(protocol:int) -> object|None:
+
+def get_protocol_class(protocol: int) -> object | None:
     return HDR_DB[protocol]['class'] if protocol in HDR_DB else None
 
-def field_is_supported(protocol:int, range_high:int, range_low:int) -> bool:
+
+def field_is_supported(protocol: int, range_high: int, range_low: int) -> bool:
     return protocol in HDR_DB and (range_high, range_low) in HDR_DB[protocol]['fields']
 
-def get_field_id(protocol:int, range_high:int, range_low:int) -> str:
+
+def get_field_id(protocol: int, range_high: int, range_low: int) -> str:
     if field_is_supported(protocol, range_high, range_low):
         return HDR_DB[protocol]['fields'][(range_high, range_low)]
     return ''
 
-def get_field_info_str(protocol:int, range_high:int, range_low:int) -> str:
+
+def get_field_info_str(protocol: int, range_high: int, range_low: int) -> str:
     if field_is_supported(protocol, range_high, range_low):
         return HDR_DB[protocol]['name'] + ' - ' + get_field_id(protocol, range_high, range_low)
     return ''