Merge pull request The-OpenROAD-Project#3272 from The-OpenROAD-Project-staging/cts_default_yaml_values

Fix default values for CTS_CLUSTER_SIZE and CTS_CLUSTER_DIAMETER

Author: maliberty

docs/user/FlowVariables.md

@@ -114,8 +114,8 @@ configuration file.
 | <a name="CTS_ARGS"></a>CTS_ARGS| Override `clock_tree_synthesis` arguments.| |
 | <a name="CTS_BUF_DISTANCE"></a>CTS_BUF_DISTANCE| Distance (in microns) between buffers.| |
 | <a name="CTS_BUF_LIST"></a>CTS_BUF_LIST| List of cells used to construct the clock tree. Overrides buffer inference.| |
-| <a name="CTS_CLUSTER_DIAMETER"></a>CTS_CLUSTER_DIAMETER| Maximum diameter (in microns) of sink cluster.| 20|
-| <a name="CTS_CLUSTER_SIZE"></a>CTS_CLUSTER_SIZE| Maximum number of sinks per cluster.| 50|
+| <a name="CTS_CLUSTER_DIAMETER"></a>CTS_CLUSTER_DIAMETER| Maximum diameter (in microns) of sink cluster.| |
+| <a name="CTS_CLUSTER_SIZE"></a>CTS_CLUSTER_SIZE| Maximum number of sinks per cluster.| |
 | <a name="CTS_LIB_NAME"></a>CTS_LIB_NAME| Name of the Liberty library to use in selecting the clock buffers.| |
 | <a name="CTS_SNAPSHOT"></a>CTS_SNAPSHOT| Creates ODB/SDC files prior to clock net and setup/hold repair.| |
 | <a name="DESIGN_NAME"></a>DESIGN_NAME| The name of the top-level module of the design.| |

flow/designs/asap7/aes-block/rules-base.json

@@ -24,7 +24,7 @@
         "compare": "<="
     },
     "cts__design__instance__count__hold_buffer": {
-        "value": 966,
+        "value": 1155,
         "compare": "<="
     },
     "globalroute__antenna_diodes_count": {

flow/designs/ihp-sg13g2/aes/rules-base.json

@@ -28,7 +28,7 @@
         "compare": "<="
     },
     "globalroute__antenna_diodes_count": {
-        "value": 6,
+        "value": 14,
         "compare": "<="
     },
     "detailedroute__route__wirelength": {
@@ -40,7 +40,7 @@
         "compare": "<="
     },
     "detailedroute__antenna__violating__nets": {
-        "value": 0,
+        "value": 1,
         "compare": "<="
     },
     "detailedroute__antenna_diodes_count": {

flow/designs/ihp-sg13g2/jpeg/rules-base.json

@@ -40,7 +40,7 @@
         "compare": "<="
     },
     "detailedroute__antenna__violating__nets": {
-        "value": 0,
+        "value": 1,
         "compare": "<="
     },
     "detailedroute__antenna_diodes_count": {

flow/designs/nangate45/swerv/rules-base.json

@@ -48,7 +48,7 @@
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -0.21,
+        "value": -0.25,
         "compare": ">="
     },
     "finish__design__instance__area": {

flow/designs/sky130hd/riscv32i/rules-base.json

@@ -28,11 +28,11 @@
         "compare": "<="
     },
     "globalroute__antenna_diodes_count": {
-        "value": 0,
+        "value": 4,
         "compare": "<="
     },
     "detailedroute__route__wirelength": {
-        "value": 301382,
+        "value": 297104,
         "compare": "<="
     },
     "detailedroute__route__drc_errors": {
@@ -44,15 +44,15 @@
         "compare": "<="
     },
     "detailedroute__antenna_diodes_count": {
-        "value": 12,
+        "value": 9,
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -1.37,
+        "value": -1.24,
         "compare": ">="
     },
     "finish__design__instance__area": {
-        "value": 90802,
+        "value": 83704,
         "compare": "<="
     },
     "finish__timing__drv__setup_violation_count": {

flow/designs/sky130hs/ibex/rules-base.json

@@ -28,7 +28,7 @@
         "compare": "<="
     },
     "globalroute__antenna_diodes_count": {
-        "value": 39,
+        "value": 20,
         "compare": "<="
     },
     "detailedroute__route__wirelength": {
@@ -44,15 +44,15 @@
         "compare": "<="
     },
     "detailedroute__antenna_diodes_count": {
-        "value": 22,
+        "value": 38,
         "compare": "<="
     },
     "finish__timing__setup__ws": {
         "value": -0.56,
         "compare": ">="
     },
     "finish__design__instance__area": {
-        "value": 268183,
+        "value": 253949,
         "compare": "<="
     },
     "finish__timing__drv__setup_violation_count": {

flow/scripts/variables.yaml

@@ -832,15 +832,13 @@ CTS_BUF_LIST:
 CTS_CLUSTER_DIAMETER:
   description: >
     Maximum diameter (in microns) of sink cluster.
-  default: 20
   stages:
     - cts
   tunable: 1
   type: float
 CTS_CLUSTER_SIZE:
   description: >
     Maximum number of sinks per cluster.
-  default: 50
   stages:
     - cts
   tunable: 1