-
Notifications
You must be signed in to change notification settings - Fork 97
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
Improve convert via to 3d #4028
Conversation
Thanks for opening a Pull Request. If you want to perform a review write a comment saying: @ansys-reviewer-bot review |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4028 +/- ##
==========================================
+ Coverage 81.47% 81.63% +0.16%
==========================================
Files 182 182
Lines 63565 63546 -19
==========================================
+ Hits 51787 51877 +90
+ Misses 11778 11669 -109 |
Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com>
Co-authored-by: Maxime Rey <87315832+MaxJPRey@users.noreply.github.com>
Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com>
Add check to exclude padstack definition using hole shape poly. I tried to use a check on self.hole_type == "NoGeometry" as it does not work I check len(self.hole_properties) which is 0 for hole shape poly and greater than 0 for all other hole shape
Could you check my last commit it seems there is a modification on file Design.py but it is not coming from me. |
Modify C:\ANSYSDEV\NoBackup\pyaedt\_unittest\example_models\TEDB\padstacks.aedb to add 4 padstack definitions with different hole shape. Modify test_128_microvias to add test when padstack definition is preserved and test padstack with different hole shape.
Fix issues on test_128_microvias
for more information, see https://pre-commit.ci
Fix issues on test_128_microvias
Update test_128_microvias with assert not for hole shpae poly
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…erent from layer name.
Improve convert via to 3D.
Add the ability to not remove the padstack definition and to set hole size to zero.