Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat: post-drt antenna repair #573

Open
wants to merge 9 commits into
base: dev
Choose a base branch
from
Open

feat: post-drt antenna repair #573

wants to merge 9 commits into from
+67 −1

Conversation

kareefardi
Copy link
Collaborator

@kareefardi kareefardi commented Oct 8, 2024 •
edited by donn
Loading

Steps

  • OpenROAD.RepairAntennasPostDRT
    • New step similar to OpenROAD.RepairAntennas but using data after detailed routing. The step fixes antenna violations and reroute affected nets.
    • New variable DRT_ANTENNA_MARGIN which is similar to GRT_ANTENNA_MARGIN

Flows

  • Classic
    • Added OpenROAD.RepairAntennasPostDRT which is gated by RUN_POST_DRT_ANTENNA_REPAIR. Disabled by default.

Depends on efabless/openlane2-ci-designs#34
Resolves #610

@kareefardi kareefardi changed the base branch from main to dev October 8, 2024 08:33
@kareefardi
Copy link
Collaborator Author

@donn Thoughts on:

  • Having the step in the classical flow
  • The additional variable DRT_ANTENNA_MARGIN

@openlane-bot
Copy link
Collaborator

openlane-bot commented Oct 8, 2024 •
edited
Loading

Metric comparisons are in beta. Please report bugs under the issues tab.

To create this report yourself, grab the metrics artifact from the CI run, extract them, and invoke python3 -m openlane.common.metrics compare-remote current --branch dev --table-verbosity ALL --table-out ./tables_all.md.

  • No changes to critical metrics were detected in analyzed designs.

Full tables ► https://gist.github.com/openlane-bot/4f0d6e6c7429cdb69f8d62cf60fcab4d

@donn donn changed the title Post drt antenna feat: post-drt antenna repair Oct 9, 2024
@kareefardi kareefardi marked this pull request as ready for review October 14, 2024 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kareefardi @openlane-bot