[Format] Fixed Python Formatting Issues

Run make format-py to fix all Python format issues.

Author: AlexandreSinger

odin_ii/regression_test/tools/odin_config_maker.py

@@ -34,7 +34,9 @@ def main(argv=None):
         print("\tDid not get any input options; use odin_config_maker.py -h for help.")
         return -1
     if options.individual is not None and options.directory is not None:
-        print("\tThe -i and -d options are mutually exclusive; use odin_config_maker.py -h for help.")
+        print(
+            "\tThe -i and -d options are mutually exclusive; use odin_config_maker.py -h for help."
+        )
         return -1
 
     # Create our Config Files

odin_ii/usefull_tools/restore_blackboxed_latches_from_blif_file.py

@@ -51,14 +51,18 @@ def main():
     if args.inplace:
         print("Inplace Restortation:")
     elif not args.restoredBlifFile:
-        print('\n\nERROR: Must Specify Either Inplace Restoration "-i,--inplace" or a file to restore to "-r\--restoredBlifFile"\nExiting...\n')
+        print(
+            '\n\nERROR: Must Specify Either Inplace Restoration "-i,--inplace" or a file to restore to "-r\--restoredBlifFile"\nExiting...\n'
+        )
         parser.print_help()
         return -1
 
     if not os.path.isfile(args.blifFileToRestore):
-        print('\n\nERROR: BLIF File to Restore "{}" doesn not exist\nExiting...\n'.format(
-            args.blifFileToRestore
-        ))
+        print(
+            '\n\nERROR: BLIF File to Restore "{}" doesn not exist\nExiting...\n'.format(
+                args.blifFileToRestore
+            )
+        )
         parser.print_help()
         return -1
 

vtr_flow/scripts/benchtracker/plotter-offline.py

@@ -226,6 +226,7 @@ def subplot_traverser(
             i.e.: y_sublist = y_raw_list[i]
         figure_name
     """
+
     # TODO: if some x,y series are all -1, then we should not create the figure
     def figure_traverser(
         self, y_sub_list, namemap, axis_left, xy_namemap, y_i, figure_name, plot_type="plot"
@@ -381,9 +382,38 @@ def main():
     data_collection = data_converter(data, ret["x"], ret["y"], filt_name_list)
 
     print("########################################")
-    print("---- Description: ----\n" + ">>\n" + "VPR benchmark experiment should have 2 types of data: \n" + "parameter: settings in for the experiment (e.g.: fc, wire length, switch block ...)\n" + "metrics: measurements from the VPR output (e.g.: min chan width, critical path delay ...)\n" + ">>\n" + "Data passed into this plotter should have already been classified into 3 axes: \n" + "one [x] axis (chosen from parameter)\n" + "multiple [y] axis (chosen from metrics)\n" + "multiple [filter] axis (all the unchosen parameters)\n" + ">>\n" + "For example, if the experiment has: \n" + "[arch, circuit, wire length, switch block, fc, min chan width, critical path delay, area, total wire length]\n" + "and you choose fc as x axis, [min chan width, critical path delay, area, total wire length] as y axes,\n" + "then filter axes are the unchosen parameters, i.e.: arch, circuit, wire length, switch block. ")
+    print(
+        "---- Description: ----\n"
+        + ">>\n"
+        + "VPR benchmark experiment should have 2 types of data: \n"
+        + "parameter: settings in for the experiment (e.g.: fc, wire length, switch block ...)\n"
+        + "metrics: measurements from the VPR output (e.g.: min chan width, critical path delay ...)\n"
+        + ">>\n"
+        + "Data passed into this plotter should have already been classified into 3 axes: \n"
+        + "one [x] axis (chosen from parameter)\n"
+        + "multiple [y] axis (chosen from metrics)\n"
+        + "multiple [filter] axis (all the unchosen parameters)\n"
+        + ">>\n"
+        + "For example, if the experiment has: \n"
+        + "[arch, circuit, wire length, switch block, fc, min chan width, critical path delay, area, total wire length]\n"
+        + "and you choose fc as x axis, [min chan width, critical path delay, area, total wire length] as y axes,\n"
+        + "then filter axes are the unchosen parameters, i.e.: arch, circuit, wire length, switch block. "
+    )
     print("#########################################")
-    print("---- Usage ----\n" + ">>\n" + "1. choose overlay axes among the filter axes (overlay axes will become legend in a single plot)\n" + '2. choose whether to whether to calculate the geo mean over the overlay axis ("merge" function)\n' + "   (Notice: you can choose as many overlay axes as you like, but when you choose merge, it will only\n" + "    calculate the geo mean over the last overlay axis. So for example, if your overlay axes are [fc, circuit],\n" + "    the merge will only get geo mean over all the circuits rather that all the (circuit,fc) combination, and \n" + "    fc will still be overlaid in the merged plot.)\n" + '3. the data after geo mean calcultion will be referred to as "gmean", and the data before the geo mean will be \n' + '   referred to as "raw", you can switch the overlay axes for both gmean data and raw data, for as many times \n' + '   as you like. But once you "merge" on a new axis, the old gmean data will be replaced by the new one, and further\n' + "   operation will be acted on only the new gmean data.")
+    print(
+        "---- Usage ----\n"
+        + ">>\n"
+        + "1. choose overlay axes among the filter axes (overlay axes will become legend in a single plot)\n"
+        + '2. choose whether to whether to calculate the geo mean over the overlay axis ("merge" function)\n'
+        + "   (Notice: you can choose as many overlay axes as you like, but when you choose merge, it will only\n"
+        + "    calculate the geo mean over the last overlay axis. So for example, if your overlay axes are [fc, circuit],\n"
+        + "    the merge will only get geo mean over all the circuits rather that all the (circuit,fc) combination, and \n"
+        + "    fc will still be overlaid in the merged plot.)\n"
+        + '3. the data after geo mean calcultion will be referred to as "gmean", and the data before the geo mean will be \n'
+        + '   referred to as "raw", you can switch the overlay axes for both gmean data and raw data, for as many times \n'
+        + '   as you like. But once you "merge" on a new axis, the old gmean data will be replaced by the new one, and further\n'
+        + "   operation will be acted on only the new gmean data."
+    )
     while 1:
         print(">>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>")
         print("available axis to overlay: ")

vtr_flow/scripts/download_titan.py

@@ -321,6 +321,7 @@ def extract_callback(members, args):
             print(tarinfo.name)
             yield tarinfo
 
+
 def get_benchmark_subdirs(args):
     """
     Decide which benchmark subdirectories to use depending on version
@@ -331,6 +332,7 @@ def get_benchmark_subdirs(args):
     else:
         return ["titan_new", "titan23", "other_benchmarks"]
 
+
 def compare_versions(version1, version2):
     """
     Compares two release versions to see which once is more recent

vtr_flow/scripts/python_libs/vtr/parse_vtr_task.py

@@ -422,7 +422,6 @@ def check_two_files(
     for (arch, circuit, script_params), _ in first_results.all_metrics().items():
         first_primary_keys.append((arch, circuit, script_params))
 
-
     # Warn about any elements in first result file that are not found in second result file
     for arch, circuit, script_params in first_primary_keys:
         if second_results.metrics(arch, circuit, script_params) is None:
@@ -443,9 +442,11 @@ def check_two_files(
                     "/".join(str((Path(config.config_dir).parent)).split("/")[-3:])
                 )
             )
-            print("Required case {}/{} missing from {} results: {}".format(
+            print(
+                "Required case {}/{} missing from {} results: {}".format(
                     arch, circuit, first_name, first_results_filepath
-                ))
+                )
+            )
             num_qor_failures += 1
             continue
 

vtr_flow/scripts/spice/run_spice.py

@@ -10,7 +10,7 @@
 my_dir = ""
 
 if len(sys.argv) < 9:
-    print (
+    print(
         "Usage: spice.py <tech_file> <tech_size> <Vdd> <P/N> <temp> <activity [hz]> <component_type> <size> <nmos_size>"
     )
     sys.exit()
@@ -37,7 +37,7 @@
 elif activity == "z":
     na = 1
 else:
-    print ("Invalid activity type\n")
+    print("Invalid activity type\n")
     sys.exit()
 
 base_dir = os.path.join(my_dir)