Skip to content

Commit 470955d

Browse files
authored
Merge pull request #1959 from The-OpenROAD-Project-staging/rule-table-format
genRuleFile.py: make the text table prettier for the commit
2 parents b992704 + c32c00b commit 470955d

File tree

1 file changed

+9
-8
lines changed

1 file changed

+9
-8
lines changed

flow/util/genRuleFile.py

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -190,6 +190,7 @@ def gen_rule_file(design_dir, update, tighten, failing, variant, golden_metrics=
190190
if len(period_list) != 1:
191191
print(f'[WARNING] Multiple clocks not supported. Will use first clock: {period_list[0]}.')
192192

193+
format_str = '| {:45} | {:8} | {:8} | {:8} |\n'
193194
change_str = ''
194195
for field, option in rules_dict.items():
195196
if field not in metrics.keys():
@@ -269,27 +270,27 @@ def gen_rule_file(design_dir, update, tighten, failing, variant, golden_metrics=
269270
and rule_value != old_rule['value'] \
270271
and compare(rule_value, old_rule['value']):
271272
UPDATE = True
272-
change_str += f"| {field} | {old_rule['value']} | "\
273-
f"{rule_value} | Tighten |\n"
273+
change_str += format_str.format(field, old_rule['value'],
274+
rule_value, 'Tighten')
274275

275276
if failing and not compare(metrics[field], old_rule['value']):
276277
UPDATE = True
277-
change_str += f"| {field} | {old_rule['value']} | " \
278-
f"{rule_value} | Failing |\n"
278+
change_str += format_str.format(field, old_rule['value'],
279+
rule_value, 'Failing')
279280

280281
if update and old_rule['value'] != rule_value:
281282
UPDATE = True
282-
change_str += f"| {field} | {old_rule['value']} | "\
283-
f"{rule_value} | Updating |\n"
283+
change_str += format.format_str.format(field, old_rule['value'],
284+
rule_value, 'Updating')
284285

285286
if not UPDATE:
286287
rule_value = old_rule['value']
287288

288289
rules[field] = dict(value=rule_value, compare=option['compare'])
289290

290291
if len(change_str) > 0:
291-
print("| Metric | Old | New | Type |")
292-
print("| ------ | --- | --- | ---- |")
292+
print(format_str.format('Metric', 'Old', 'New', 'Type'))
293+
print(format_str.format('------', '---', '---', '----'))
293294
print(change_str)
294295

295296
with open(rules_file, 'w') as f:

0 commit comments

Comments
 (0)