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

REFACTOR: Optimize Edb initialization for die designs #966

Merged
merged 2 commits into from
Jan 13, 2025

Conversation

SMoraisAnsys
Copy link
Collaborator

Duplicates PR #962 which was created from a fork.


Several times, die designs don't have any component in them.
The code changes made in this PR signifiantly reduces the Component initialization time in this case which thereby reduces the Edb initialization time by ~50% especially when there are >100k Groups in the layout.

This should also speed up EDB initialization time when there are 1000s of components in the design (Ex: Deep trench capacitors in the IC).

Several times die designs don't have any component defined. The changes
in code are made to reduce the Component initialization time in this
case which significantly reduces the Edb initialization time by ~50%
especially when there are >100k Groups in the layout.

(cherry picked from commit 40cc8f6)
@github-actions github-actions bot added the grpc-transition Changes that need to be included in gRPC transition label Jan 8, 2025
@codecov-commenter
Copy link

codecov-commenter commented Jan 8, 2025

Codecov Report

Attention: Patch coverage is 95.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 82.68%. Comparing base (7bef7c3) to head (d931ac7).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #966      +/-   ##
==========================================
- Coverage   82.70%   82.68%   -0.03%     
==========================================
  Files         160      160              
  Lines       21121    21098      -23     
==========================================
- Hits        17468    17444      -24     
- Misses       3653     3654       +1     

Copy link
Collaborator

@hui-zhou-a hui-zhou-a left a comment

Choose a reason for hiding this comment

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

LGTM

@hui-zhou-a hui-zhou-a changed the title Optimize Edb initialization for die designs REFACTOR: Optimize Edb initialization for die designs Jan 8, 2025
@skandak-ansys skandak-ansys merged commit fc0c8fa into main Jan 13, 2025
30 checks passed
@skandak-ansys skandak-ansys deleted the refactor/components-init branch January 13, 2025 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
grpc-transition Changes that need to be included in gRPC transition
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants