Skip to content

updated CTS_BUF_DISTANCE handling + AT genMetric.py call fix #3115

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed

Conversation

jeffng-or
Copy link
Contributor

Merge of #3106 and part of #3108.

  • Fixes CTS_BUF_DISTANCE flow variable handling
  • Fixed call to genMetrics.py in AutoTuner

@luarss , @vvbandeira , @arthurjolo FYI.

Removed sky130hd gcd from sample iteration test

Signed-off-by: Jeff Ng <jeffng@precisioninno.com>
@jeffng-or jeffng-or force-pushed the at-cts-arg-handling-1 branch from 5ad987b to 6836b18 Compare April 28, 2025 23:29
@jeffng-or
Copy link
Contributor Author

@luarss , @vvbandeira FYI. I removed sky130hd gcd from the sample iteration test because it continually fails with sample == 1. The AT can't find a successful run with only one sample, so it's a fragile test.

@luarss
Copy link
Contributor

luarss commented Apr 29, 2025

Yep that is fine - I have pushed a fix in #3025 to be more tolerant of parameter tuning failure (error code 16), so that is a more long term fix

@jeffng-or
Copy link
Contributor Author

@luarss , @vvbandeira FYI. I removed sky130hd gcd from the sample iteration test because it continually fails with sample == 1. The AT can't find a successful run with only one sample, so it's a fragile test.

It looks like the failure is a seg fault in DPL. I'm able to repro locally, so will file an issue. Closing this PR and will send up a separate PR for the genMetrics.py fix, so that we can get the AT working again.

@jeffng-or jeffng-or closed this Apr 29, 2025
@jeffng-or jeffng-or deleted the at-cts-arg-handling-1 branch May 9, 2025 15:49
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.

2 participants