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.

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

DRC input "METAL (17/0)" filter does not work #1887

Closed
Fusion86 opened this issue Oct 11, 2024 · 1 comment
Closed

DRC input "METAL (17/0)" filter does not work #1887

Fusion86 opened this issue Oct 11, 2024 · 1 comment
Assignees
Labels
Milestone

Comments

@Fusion86
Copy link

If I read the docs correctly then these two statements should produce the same result. However, the second one throws an error.

active = input(RBA::LayerInfo::new(43, 1, "active"))
active = input("active (43/1)")

image

This code is inside an LVS macro, but as far as I understand this code is the same as what the DRC engine runs?
Could also be that this is intentional behavior, but then the docs might be wrong?

It works fine if you only specify the layer name without the space+numbers (set inside 'Edit Layer Specification' which is somehow different from the right-click->Rename name?)

@klayoutmatthias
Copy link
Collaborator

You're right. Thanks for pointing that out. The documentation was not updated when introducing multi-layer inputs.

The string-only versions provide filters that can collect multiple layout layers into one DRC layer, e.g.

# combines all datatypes of layer 1 in one DRC layer
l1 = input("1/*") 
# combines layer 1/0 and 10/0
l1_and_10 = input("1/0", "10/0")

So your case would be:

active = input("active", "43/1")

However, my proposal is to use layer/datatypes only if you're working with GDS or OASIS. In GDS there is no layer name at all and OASIS is weird - the name can refer to multiple layers or not be present at all. The only reliable layer coordinate is layer/datatype.

Layer names are provided mainly for formats that only use named layers, such as CIF and DXF.

Matthias

@klayoutmatthias klayoutmatthias self-assigned this Oct 12, 2024
klayoutmatthias pushed a commit that referenced this issue Oct 12, 2024
@klayoutmatthias klayoutmatthias modified the milestones: 0.29.7, 0.29.8 Oct 12, 2024
klayoutmatthias added a commit that referenced this issue Oct 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants