-
Notifications
You must be signed in to change notification settings - Fork 126
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
padstack name #3814
padstack name #3814
Conversation
Thanks for opening a Pull Request. If you want to perform a review write a comment saying: @ansys-reviewer-bot review |
Codecov Report
@@ Coverage Diff @@
## main #3814 +/- ##
==========================================
- Coverage 81.12% 81.11% -0.01%
==========================================
Files 180 180
Lines 62257 62254 -3
==========================================
- Hits 50504 50498 -6
- Misses 11753 11756 +3 |
Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com>
I am not sure sure we can change the pin naming (e.g component.pin name). This will for sure break current flows. I remember we had to sync EDB pin name with AEDT ones. @maxcapodi78 what do you think ? |
@maxcapodi78 @svandenb-dev there is a change in the padstack name. It may break customer code, but I think it is better to correct the way how the padstack name is returned in the lone run.