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

connect (in flat mode) + multiple top cells give an error message #1594

Closed
klayoutmatthias opened this issue Jan 18, 2024 · 0 comments · Fixed by #1596
Closed

connect (in flat mode) + multiple top cells give an error message #1594

klayoutmatthias opened this issue Jan 18, 2024 · 0 comments · Fixed by #1596
Assignees
@klayoutmatthias
Labels
bug
Milestone
0.28.16

Comments

@klayoutmatthias
Copy link
Collaborator

klayoutmatthias commented Jan 18, 2024 •
edited
Loading

For details see here: https://www.klayout.de/forum/discussion/2455/short-testing-connect-issue#latest

The issue happens in flat mode.

The problem is in _drc_netter.rb:705

layout = @engine.source.layout
@l2n = RBA::LayoutToNetlist::new(layout.top_cell.name, layout.dbu)

The name of the cell should be taken from "source".
@klayoutmatthias klayoutmatthias changed the title connect + connect (in flat mode) + multiple top cells give an error message Jan 19, 2024
klayoutmatthias pushed a commit that referenced this issue Jan 20, 2024
Fixed issue #1594
d8c00de
@klayoutmatthias klayoutmatthias self-assigned this Jan 20, 2024
@klayoutmatthias klayoutmatthias added this to the 0.28.16 milestone Jan 20, 2024
@klayoutmatthias klayoutmatthias linked a pull request Feb 11, 2024 that will close this issue
Fixed issue #1594 #1596
Merged
klayoutmatthias added a commit that referenced this issue Feb 11, 2024
@klayoutmatthias
Fixed issue #1594 (#1596)
3ab442b 
Co-authored-by: Matthias Koefferlein <matthias@klayout.de>
klayoutmatthias added a commit that referenced this issue Feb 11, 2024
@klayoutmatthias
Fixed issue #1594 (#1596)
386b567 
Co-authored-by: Matthias Koefferlein <matthias@klayout.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@klayoutmatthias klayoutmatthias

Labels
bug
Projects
None yet
Milestone
0.28.16
Development

Successfully merging a pull request may close this issue.

Fixed issue #1594 KLayout/klayout
1 participant
@klayoutmatthias