consistency

efabless · Feb 13, 2025 · f9079e9 · f9079e9
1 parent 58042ea
commit f9079e9
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 14 deletions.
diff --git a/openlane/scripts/openroad/drt.tcl b/openlane/scripts/openroad/drt.tcl
@@ -67,7 +67,7 @@ drt_run $i {*}$drt_args
 incr i
 
 if { ![info exists ::env(DIODE_CELL)] } {
-    puts "\[INFO\] Skipping post-DRT antenna repair: DIODE_CELL not set."
+    puts "\[INFO\] Skipping post-DRT antenna repair: 'DIODE_CELL' not set."
 } elseif { $::env(DRT_ANTENNA_REPAIR_ITERS) == 0 } {
     puts "\[INFO\] Skipping post-DRT antenna repair: DRT_ANTENNA_REPAIR_ITERS set to 0."
 } else {

diff --git a/openlane/steps/cvc_rv.py b/openlane/steps/cvc_rv.py
@@ -165,5 +165,5 @@ def run(self, state_in: State, **kwargs) -> Tuple[ViewsUpdate, MetricsUpdate]:
             )
             return {}, {}
         except CVCNoSupport as e:
-            self.warn(f"Could not run CVC: {e}. Skipping…")
+            self.warn(f"Could not run CVC: {e}. Skipping '{self.id}'…")
             return {}, {}
diff --git a/openlane/steps/odb.py b/openlane/steps/odb.py
@@ -178,7 +178,7 @@ def get_command(self) -> List[str]:
 
     def run(self, state_in, **kwargs) -> Tuple[ViewsUpdate, MetricsUpdate]:
         if not self.get_cells():
-            info("No cells provided, skipping…")
+            info(f"No cells provided, skipping '{self.id}'…")
             return {}, {}
         return super().run(state_in, **kwargs)
 
@@ -242,7 +242,7 @@ def get_command(self) -> List[str]:
 
     def run(self, state_in, **kwargs) -> Tuple[ViewsUpdate, MetricsUpdate]:
         if self.config["FP_DEF_TEMPLATE"] is None:
-            info("No DEF template provided, skipping…")
+            info(f"No DEF template provided, skipping '{self.id}'…")
             return {}, {}
 
         views_updates, metrics_updates = super().run(state_in, **kwargs)
@@ -657,7 +657,7 @@ def get_command(self) -> List[str]:
 
     def run(self, state_in, **kwargs) -> Tuple[ViewsUpdate, MetricsUpdate]:
         if self.config["FP_PIN_ORDER_CFG"] is None:
-            info("No custom floorplan file configured, skipping…")
+            info(f"No custom floorplan file configured, skipping '{self.id}'…")
             return {}, {}
         return super().run(state_in, **kwargs)
 
@@ -715,11 +715,11 @@ def get_command(self) -> List[str]:
 
     def run(self, state_in: State, **kwargs) -> Tuple[ViewsUpdate, MetricsUpdate]:
         if self.config["DIODE_ON_PORTS"] == "none":
-            info("'DIODE_ON_PORTS' is set to 'none': skipping…")
+            info(f"'DIODE_ON_PORTS' is set to 'none': skipping '{self.id}'…")
             return {}, {}
 
         if self.config["DIODE_CELL"] is None:
-            info("DIODE_CELL not set. Skipping…")
+            info(f"'DIODE_CELL' not set. Skipping '{self.id}'…")
             return {}, {}
 
         if self.config["GPL_CELL_PADDING"] == 0:
@@ -759,10 +759,10 @@ class DiodesOnPorts(CompositeStep):
 
     def run(self, state_in: State, **kwargs) -> Tuple[ViewsUpdate, MetricsUpdate]:
         if self.config["DIODE_ON_PORTS"] == "none":
-            info("'DIODE_ON_PORTS' is set to 'none': skipping…")
+            info(f"'DIODE_ON_PORTS' is set to 'none': skipping '{self.id}'…")
             return {}, {}
         if self.config["DIODE_CELL"] is None:
-            info("DIODE_CELL not set. Skipping…")
+            info(f"'DIODE_CELL' not set. Skipping '{self.id}'…")
             return {}, {}
         return super().run(state_in, **kwargs)
 
@@ -834,7 +834,7 @@ def run(self, state_in: State, **kwargs) -> Tuple[ViewsUpdate, MetricsUpdate]:
             )
 
         if self.config["DIODE_CELL"] is None:
-            info("DIODE_CELL not set. Skipping…")
+            info(f"'DIODE_CELL' not set. Skipping '{self.id}'…")
             return {}, {}
 
         return super().run(state_in, **kwargs)
@@ -873,7 +873,7 @@ class HeuristicDiodeInsertion(CompositeStep):
 
     def run(self, state_in: State, **kwargs) -> Tuple[ViewsUpdate, MetricsUpdate]:
         if self.config["DIODE_CELL"] is None:
-            info("DIODE_CELL not set. Skipping…")
+            info(f"'DIODE_CELL' not set. Skipping '{self.id}'…")
             return {}, {}
         return super().run(state_in, **kwargs)
 
@@ -963,6 +963,6 @@ def get_command(self) -> List[str]:
 
     def run(self, state_in, **kwargs) -> Tuple[ViewsUpdate, MetricsUpdate]:
         if self.config["MANUAL_GLOBAL_PLACEMENTS"] is None:
-            info("'MANUAL_GLOBAL_PLACEMENTS' not set, skipping…")
+            info(f"'MANUAL_GLOBAL_PLACEMENTS' not set, skipping '{self.id}'…")
             return {}, {}
         return super().run(state_in, **kwargs)
diff --git a/openlane/steps/openroad.py b/openlane/steps/openroad.py
@@ -1673,7 +1673,7 @@ class RepairAntennas(CompositeStep):
 
     def run(self, state_in: State, **kwargs) -> Tuple[ViewsUpdate, MetricsUpdate]:
         if self.config["DIODE_CELL"] is None:
-            info("DIODE_CELL not set. Skipping…")
+            info(f"'DIODE_CELL' not set. Skipping '{self.id}'…")
             return {}, {}
 
         return super().run(state_in, **kwargs)

diff --git a/openlane/steps/yosys.py b/openlane/steps/yosys.py
@@ -297,7 +297,7 @@ def run(self, state_in: State, **kwargs) -> Tuple[ViewsUpdate, MetricsUpdate]:
             )
         else:
             info(
-                f"PDK {self.config['PDK']} is not supported by the EQY step. Skipping…"
+                f"PDK {self.config['PDK']} is not supported by the EQY step. Skipping '{self.id}'…"
             )
             return {}, {}
-Original file line number
+Diff line change
@@ Expand Up @@
                 )
             else:
                 info(
-                    f"PDK {self.config['PDK']} is not supported by the EQY step. Skipping…"
+                    f"PDK {self.config['PDK']} is not supported by the EQY step. Skipping '{self.id}'…"
                 )
                 return {}, {}
@@ Expand Down @@