Skip to content

Commit f50270a

Browse files
authored
Merge pull request #1417 from The-OpenROAD-Project-staging/secure-metrics-relax
Relax CI metrics ci: only look at final timing ci: remove brittle drv limit metrics ci: make setup violations dependent on design size ci: increase buffer and hold violation limits ci: propagate rules changes Bump OR submodule util: add check to not update to failing value ci: fix public failure
2 parents 021e724 + 20830d0 commit f50270a

File tree

83 files changed

+818
-3397
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

83 files changed

+818
-3397
lines changed
Lines changed: 9 additions & 41 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,14 @@
11
{
22
"synth__design__instance__area__stdcell": {
3-
"value": 2576,
3+
"value": 2576.15,
44
"compare": "<="
55
},
66
"constraints__clocks__count": {
77
"value": 1,
88
"compare": "=="
99
},
1010
"placeopt__design__instance__area": {
11-
"value": 2871,
11+
"value": 2869,
1212
"compare": "<="
1313
},
1414
"placeopt__design__instance__count__stdcell": {
@@ -19,36 +19,16 @@
1919
"value": 0,
2020
"compare": "=="
2121
},
22-
"cts__timing__setup__ws": {
23-
"value": -100.0,
24-
"compare": ">="
25-
},
26-
"cts__timing__setup__ws__pre_repair": {
27-
"value": -250.75,
28-
"compare": ">="
29-
},
30-
"cts__timing__setup__ws__post_repair": {
31-
"value": -250.75,
32-
"compare": ">="
33-
},
3422
"cts__design__instance__count__setup_buffer": {
35-
"value": 2717,
23+
"value": 2566,
3624
"compare": "<="
3725
},
3826
"cts__design__instance__count__hold_buffer": {
39-
"value": 1105,
27+
"value": 2211,
4028
"compare": "<="
4129
},
42-
"globalroute__timing__clock__slack": {
43-
"value": -79.41,
44-
"compare": ">="
45-
},
46-
"globalroute__timing__setup__ws": {
47-
"value": -79.41,
48-
"compare": ">="
49-
},
5030
"detailedroute__route__wirelength": {
51-
"value": 116087,
31+
"value": 126061,
5232
"compare": "<="
5333
},
5434
"detailedroute__route__drc_errors": {
@@ -60,31 +40,19 @@
6040
"compare": ">="
6141
},
6242
"finish__design__instance__area": {
63-
"value": 3035,
43+
"value": 3287,
6444
"compare": "<="
6545
},
66-
"finish__timing__drv__max_slew_limit": {
67-
"value": -0.2,
68-
"compare": ">="
69-
},
70-
"finish__timing__drv__max_fanout_limit": {
71-
"value": -0.2,
72-
"compare": ">="
73-
},
74-
"finish__timing__drv__max_cap_limit": {
75-
"value": -0.2,
76-
"compare": ">="
77-
},
7846
"finish__timing__drv__setup_violation_count": {
79-
"value": 44,
47+
"value": 1105,
8048
"compare": "<="
8149
},
8250
"finish__timing__drv__hold_violation_count": {
83-
"value": 102,
51+
"value": 106,
8452
"compare": "<="
8553
},
8654
"finish__timing__wns_percent_delay": {
87-
"value": -10.0,
55+
"value": -16.45,
8856
"compare": ">="
8957
}
9058
}

flow/designs/asap7/aes_lvt/rules-base.json

Lines changed: 10 additions & 42 deletions
Original file line numberDiff line numberDiff line change
@@ -1,86 +1,54 @@
11
{
22
"synth__design__instance__area__stdcell": {
3-
"value": 2576,
3+
"value": 2614.16,
44
"compare": "<="
55
},
66
"constraints__clocks__count": {
77
"value": 1,
88
"compare": "=="
99
},
1010
"placeopt__design__instance__area": {
11-
"value": 2871,
11+
"value": 2870,
1212
"compare": "<="
1313
},
1414
"placeopt__design__instance__count__stdcell": {
15-
"value": 25423,
15+
"value": 25671,
1616
"compare": "<="
1717
},
1818
"detailedplace__design__violations": {
1919
"value": 0,
2020
"compare": "=="
2121
},
22-
"cts__timing__setup__ws": {
23-
"value": -100.0,
24-
"compare": ">="
25-
},
26-
"cts__timing__setup__ws__pre_repair": {
27-
"value": -250.75,
28-
"compare": ">="
29-
},
30-
"cts__timing__setup__ws__post_repair": {
31-
"value": -250.75,
32-
"compare": ">="
33-
},
3422
"cts__design__instance__count__setup_buffer": {
35-
"value": 2858,
23+
"value": 2232,
3624
"compare": "<="
3725
},
3826
"cts__design__instance__count__hold_buffer": {
39-
"value": 1105,
27+
"value": 2232,
4028
"compare": "<="
4129
},
42-
"globalroute__timing__clock__slack": {
43-
"value": -58.79,
44-
"compare": ">="
45-
},
46-
"globalroute__timing__setup__ws": {
47-
"value": -58.79,
48-
"compare": ">="
49-
},
5030
"detailedroute__route__wirelength": {
51-
"value": 116087,
31+
"value": 117485,
5232
"compare": "<="
5333
},
5434
"detailedroute__route__drc_errors": {
5535
"value": 0,
5636
"compare": "<="
5737
},
5838
"finish__timing__setup__ws": {
59-
"value": -46.45,
39+
"value": -8.46,
6040
"compare": ">="
6141
},
6242
"finish__design__instance__area": {
63-
"value": 3035,
43+
"value": 2969,
6444
"compare": "<="
6545
},
66-
"finish__timing__drv__max_slew_limit": {
67-
"value": -0.2,
68-
"compare": ">="
69-
},
70-
"finish__timing__drv__max_fanout_limit": {
71-
"value": -0.2,
72-
"compare": ">="
73-
},
74-
"finish__timing__drv__max_cap_limit": {
75-
"value": -0.2,
76-
"compare": ">="
77-
},
7846
"finish__timing__drv__setup_violation_count": {
79-
"value": 10,
47+
"value": 1116,
8048
"compare": "<="
8149
},
8250
"finish__timing__drv__hold_violation_count": {
83-
"value": 36,
51+
"value": 128,
8452
"compare": "<="
8553
},
8654
"finish__timing__wns_percent_delay": {

flow/designs/asap7/ethmac/rules-base.json

Lines changed: 8 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -1,50 +1,34 @@
11
{
22
"synth__design__instance__area__stdcell": {
3-
"value": 8662,
3+
"value": 8702.92,
44
"compare": "<="
55
},
66
"constraints__clocks__count": {
77
"value": 3,
88
"compare": "=="
99
},
1010
"placeopt__design__instance__area": {
11-
"value": 8744,
11+
"value": 8754,
1212
"compare": "<="
1313
},
1414
"placeopt__design__instance__count__stdcell": {
15-
"value": 66415,
15+
"value": 66457,
1616
"compare": "<="
1717
},
1818
"detailedplace__design__violations": {
1919
"value": 0,
2020
"compare": "=="
2121
},
22-
"cts__timing__setup__ws": {
23-
"value": -127.74,
24-
"compare": ">="
25-
},
26-
"cts__timing__setup__ws__pre_repair": {
27-
"value": -214.42,
28-
"compare": ">="
29-
},
30-
"cts__timing__setup__ws__post_repair": {
31-
"value": -214.42,
32-
"compare": ">="
33-
},
3422
"cts__design__instance__count__setup_buffer": {
35-
"value": 2888,
23+
"value": 5779,
3624
"compare": "<="
3725
},
3826
"cts__design__instance__count__hold_buffer": {
39-
"value": 2888,
27+
"value": 5779,
4028
"compare": "<="
4129
},
42-
"globalroute__timing__setup__ws": {
43-
"value": -47.32,
44-
"compare": ">="
45-
},
4630
"detailedroute__route__wirelength": {
47-
"value": 250821,
31+
"value": 253182,
4832
"compare": "<="
4933
},
5034
"detailedroute__route__drc_errors": {
@@ -59,24 +43,12 @@
5943
"value": 8802,
6044
"compare": "<="
6145
},
62-
"finish__timing__drv__max_slew_limit": {
63-
"value": -2.9,
64-
"compare": ">="
65-
},
66-
"finish__timing__drv__max_fanout_limit": {
67-
"value": -0.2,
68-
"compare": ">="
69-
},
70-
"finish__timing__drv__max_cap_limit": {
71-
"value": -0.2,
72-
"compare": ">="
73-
},
7446
"finish__timing__drv__setup_violation_count": {
75-
"value": 10,
47+
"value": 2889,
7648
"compare": "<="
7749
},
7850
"finish__timing__drv__hold_violation_count": {
79-
"value": 10,
51+
"value": 100,
8052
"compare": "<="
8153
},
8254
"finish__timing__wns_percent_delay": {

flow/designs/asap7/ethmac_lvt/rules-base.json

Lines changed: 9 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -8,43 +8,27 @@
88
"compare": "=="
99
},
1010
"placeopt__design__instance__area": {
11-
"value": 8727,
11+
"value": 8730,
1212
"compare": "<="
1313
},
1414
"placeopt__design__instance__count__stdcell": {
15-
"value": 65673,
15+
"value": 65675,
1616
"compare": "<="
1717
},
1818
"detailedplace__design__violations": {
1919
"value": 0,
2020
"compare": "=="
2121
},
22-
"cts__timing__setup__ws": {
23-
"value": -118.27,
24-
"compare": ">="
25-
},
26-
"cts__timing__setup__ws__pre_repair": {
27-
"value": -157.49,
28-
"compare": ">="
29-
},
30-
"cts__timing__setup__ws__post_repair": {
31-
"value": -157.49,
32-
"compare": ">="
33-
},
3422
"cts__design__instance__count__setup_buffer": {
35-
"value": 2855,
23+
"value": 5711,
3624
"compare": "<="
3725
},
3826
"cts__design__instance__count__hold_buffer": {
39-
"value": 2855,
27+
"value": 5711,
4028
"compare": "<="
4129
},
42-
"globalroute__timing__setup__ws": {
43-
"value": -35.21,
44-
"compare": ">="
45-
},
4630
"detailedroute__route__wirelength": {
47-
"value": 250821,
31+
"value": 267606,
4832
"compare": "<="
4933
},
5034
"detailedroute__route__drc_errors": {
@@ -56,31 +40,19 @@
5640
"compare": ">="
5741
},
5842
"finish__design__instance__area": {
59-
"value": 8774,
43+
"value": 8777,
6044
"compare": "<="
6145
},
62-
"finish__timing__drv__max_slew_limit": {
63-
"value": -1.22,
64-
"compare": ">="
65-
},
66-
"finish__timing__drv__max_fanout_limit": {
67-
"value": -0.2,
68-
"compare": ">="
69-
},
70-
"finish__timing__drv__max_cap_limit": {
71-
"value": -0.2,
72-
"compare": ">="
73-
},
7446
"finish__timing__drv__setup_violation_count": {
75-
"value": 10,
47+
"value": 2855,
7648
"compare": "<="
7749
},
7850
"finish__timing__drv__hold_violation_count": {
79-
"value": 10,
51+
"value": 100,
8052
"compare": "<="
8153
},
8254
"finish__timing__wns_percent_delay": {
8355
"value": -10.0,
8456
"compare": ">="
8557
}
86-
}
58+
}

0 commit comments

Comments
 (0)