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!: convert OR detailed router drc file to klayout xml #607

Merged
merged 12 commits into from
Feb 13, 2025
Merged

feat!: convert OR detailed router drc file to klayout xml #607

merged 12 commits into from
Feb 13, 2025

Conversation

kareefardi
Copy link
Collaborator

@kareefardi kareefardi commented Nov 28, 2024 •
edited
Loading

Fixes #527

@kareefardi
Squashed commit of the following:
58bafed 
commit e7a0e71
Author: Kareem Farid <[email protected]>
Date:   Sun Nov 3 13:22:53 2024 +0200

    lint

    Signed-off-by: Kareem Farid <[email protected]>

commit 9e25b72
Author: Kareem Farid <[email protected]>
Date:   Sun Nov 3 13:11:51 2024 +0200

    update changelog, Verilator.Lint, Yosys.JsonHeader

    Signed-off-by: Kareem Farid <[email protected]>

commit 36bca9b
Author: Kareem Farid <[email protected]>
Date:   Sun Nov 3 12:52:59 2024 +0200

    bugfix: allow unsetting variables in configuration files

    Signed-off-by: Kareem Farid <[email protected]>

commit b0246e6
Author: Kareem Farid <[email protected]>
Date:   Tue Oct 29 15:29:20 2024 +0300

    hotfix: incorrect error message when substiuting with a nonexistent step

commit 19109e1
Author: Mohamed Gaber <[email protected]>
Date:   Sat Oct 26 20:12:12 2024 +0300

    bugfix: odb steps dropping PYTHONPATH (#592)

    * `Odb.*`
      * Fixed OpenROAD dropping user-set `PYTHONPATH` values.

    ## Tool Updates

    * Use `NIX_PYTHONPATH` instead of `PYTHONPATH` in Docker and devshells
      to avoid collisions with user-set `PYTHONPATH` variables.

commit 627a5ab
Author: Mohamed Gaber <[email protected]>
Date:   Thu Oct 24 11:03:55 2024 +0300

    nix: make flake.createOpenLane shell more ergonomic

    ## Tool Updates
    * `flake.createOpenLaneShell` now gets OpenLane from `python3.pkgs`.
    * Fixed issue with `flake.createOpenLaneShell` where plugins would not get
      included due to an operator precedence issue.

commit 8451508
Author: Mohamed Gaber <[email protected]>
Date:   Thu Oct 24 10:57:58 2024 +0300

    hotfix: regression in nix/create-shell.nix

commit 1bfa43d
Author: Mohamed Gaber <[email protected]>
Date:   Mon Oct 21 21:05:05 2024 +0300

    Temporarily revert Colab's OpenLane version until it's fixed

Signed-off-by: Kareem Farid <[email protected]>
@kareefardi
feat: convert OR detailed router drc file to klayout xml
af4a258 
Signed-off-by: Kareem Farid <[email protected]>
@openlane-bot
Copy link
Collaborator

openlane-bot commented Nov 28, 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/6e7d2da424e6f455e046db1e98a3fe9a

donn
donn reviewed Nov 29, 2024
View reviewed changes
openlane/common/drc.py Outdated Show resolved Hide resolved
donn
donn reviewed Nov 29, 2024
View reviewed changes
openlane/common/drc.py Outdated Show resolved Hide resolved
@kareefardi
Copy link
Collaborator Author

Delayed till OpenLane 3. For cleaner implementation with API break

@kareefardi kareefardi marked this pull request as draft December 11, 2024 09:09
@kareefardi kareefardi added this to the 3.0.0 milestone Dec 11, 2024
@kareefardi
Merge remote-tracking branch 'origin/dev' into tr-to-klayoutxml
4893817 
Signed-off-by: Kareem Farid <[email protected]>
@kareefardi
make BoundingBox dataclass
e8aaf69 
update regex to new openroad format

Signed-off-by: Kareem Farid <[email protected]>
@kareefardi
update changelog
0eecd38 
Signed-off-by: Kareem Farid <[email protected]>
@kareefardi kareefardi marked this pull request as ready for review February 12, 2025 14:19
@kareefardi kareefardi requested a review from donn February 12, 2025 14:19
@kareefardi kareefardi changed the title feat: convert OR detailed router drc file to klayout xml feat!: convert OR detailed router drc file to klayout xml Feb 12, 2025
donn
donn approved these changes Feb 13, 2025
View reviewed changes
Copy link
Member

@donn donn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wrote a test, overall looks good

@donn donn enabled auto-merge (squash) February 13, 2025 13:26
@donn donn disabled auto-merge February 13, 2025 14:57
@donn donn merged commit 7e30ca1 into dev Feb 13, 2025
109 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@donn donn donn approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
3.0.0
Development

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