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

LEF/DEF topics #1307

Closed
klayoutmatthias opened this issue Feb 27, 2023 · 5 comments
Closed

LEF/DEF topics #1307

klayoutmatthias opened this issue Feb 27, 2023 · 5 comments

Comments

@klayoutmatthias
Copy link
Collaborator

This ticket is collecting some feedback regarding LEF/DEF implementation:

  1. IOTIMINGS section needs to be parsed / skipped:
DESIGN test ;
IOTIMINGS 206 ;
- ( PIN a ) + RISE SLEWRATE 500 500 + FALL SLEWRATE 500 500 ;
- ( PIN b ) + RISE SLEWRATE 500 500 + FALL SLEWRATE 500 500 ;
- ( PIN c ) + RISE SLEWRATE 500 500 + FALL SLEWRATE 500 500
  + CAPACITANCE 250000 ;
END IOTIMINGS
END DESIGN
  1. COMPONENTS + UNPLACED appears to potentially have a point and rotation in some flavors -> in that case, place it.

  2. PIN labels are taken from NET, not PIN name

  3. reading hierarchical def files (blending?) - If there is no lef MACRO for one of the def files, use the def DIEAREA bounding box instead of the lef SIZE/ORIGIN

NOTE: right now, KLayout does not support hierarchical DEF as is. 'blending' is an option to blindly merge files. The included files do now know about details of the cell calls and vice versa.

klayoutmatthias pushed a commit that referenced this issue Aug 3, 2023
…importer, generating pin names on pin name property + generating net names on net name property for pin shapes.
klayoutmatthias added a commit that referenced this issue Aug 6, 2023
Fixed part of issue #1307: using pin names for the pin labels in DEF …
@stefanottili
Copy link

stefanottili commented Sep 9, 2023

I just ran into the " + UNPLACED ( x y ) orient " components are dropped issue again.
It would be nice if strm2oas would issue a warning and that these are placed as if they're + PLACED, + FIXED or + COVER

@klayoutmatthias
Copy link
Collaborator Author

I'm not sure if there is anything left. Close that issue?

@stefanottili
Copy link

Did you implement 1) ?
ignore 4). There is a warning about missing lef, good enough.

@klayoutmatthias
Copy link
Collaborator Author

Right, IOTIMINGS is not implemented. I couldn't find a reference to that in the 5.8 doc.

Matthias

@stefanottili
Copy link

IOTIMINGS seem to be so ancient that this issue can be closed.

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

No branches or pull requests

2 participants