OpenROAD icon indicating copy to clipboard operation
OpenROAD copied to clipboard

gpl update log messages and README for routability driven mode

Open gudeh opened this issue 1 year ago • 6 comments

Improve log descriptions to include detailed information about the stages of routability-driven mode. Align log output values for better readability. Add a brief explanation of the routability-driven mode in the readme file.

gudeh avatar Apr 26 '24 23:04 gudeh

This modifications should help to address the requests made in issue #4974.

gudeh avatar Apr 26 '24 23:04 gudeh

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar Apr 26 '24 23:04 github-actions[bot]

As long as you're changing this, can I make a request to print areas in um^2 instead of DBU^2? Perhaps with 2 decimal places. The DBU number is almost never useful to anyone. Same for length in um instead of DBU.

rovinski avatar Apr 29 '24 04:04 rovinski

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar May 01 '24 01:05 github-actions[bot]

[INFO GPL-0012] DieAreaLxLy: ( 0.00, 0.00) [INFO GPL-0013] DieAreaUxUy: ( 36.29, 35.84) [INFO GPL-0014] CoreAreaLxLy: ( 1.14, 1.12) [INFO GPL-0015] CoreAreaUxUy: ( 35.15, 34.72) [INFO GPL-0016] CoreArea: 1142.74

  1. LxLy ? UxUy ?

I find lb for left bottom and rt for right top much more meaningful.

  1. less is more, why not

[INFO GPL-0012] DieArea: ( 0.00 0.00 ) ( 36.29 35.84 ) = … [INFO GPL-0014] CoreArea: ( 1.14 1.12 ) ( 35.15 34.72 ) = 1142.74

  1. a space between the number and brackets, no need for a comma. This would make it easier to read for scripting, grep, awk …

stefanottili avatar May 01 '24 02:05 stefanottili

[INFO GPL-0012] DieAreaLxLy: ( 0.00, 0.00) [INFO GPL-0013] DieAreaUxUy: ( 36.29, 35.84) [INFO GPL-0014] CoreAreaLxLy: ( 1.14, 1.12) [INFO GPL-0015] CoreAreaUxUy: ( 35.15, 34.72) [INFO GPL-0016] CoreArea: 1142.74

  1. LxLy ? UxUy ?

I find lb for left bottom and rt for right top much more meaningful.

  1. less is more, why not

[INFO GPL-0012] DieArea: ( 0.00 0.00 ) ( 36.29 35.84 ) = … [INFO GPL-0014] CoreArea: ( 1.14 1.12 ) ( 35.15 34.72 ) = 1142.74

  1. a space between the number and brackets, no need for a comma. This would make it easier to read for scripting, grep, awk …

Thanks for the suggestions! I will apply them.

gudeh avatar May 01 '24 02:05 gudeh

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar May 07 '24 22:05 github-actions[bot]

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar May 08 '24 15:05 github-actions[bot]

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar May 08 '24 15:05 github-actions[bot]