Skip to content

Conversation

ericastor
Copy link
Collaborator

The metrics extraction logic seems to have had a few bugs; some may be longstanding, but others may have been rot.

The overall effect was that WNS and TNS were incorrectly reported as 0, and that (non-timing-repair) buffer area would be reported as equal to the buffer count. These are now fixed.

@ericastor ericastor requested a review from mikesinouye August 14, 2025 16:37
@mikesinouye
Copy link
Collaborator

I think the issue here is that the OSS version of openroad used in this repo is very old, and the syntax used here is compatible with that version. We have not imported this repo directly in a while due to the skew.

We are very close to being able to bump the version of openroad used here - bazel build is very close to complete in openroad main.

@maliberty
Copy link
Contributor

We should move anything you need to the metrics file which is json and doesn't require any awk scraping.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants