-
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
Create new function to import GDS file in HFSS #4142
Conversation
Thanks for opening a Pull Request. If you want to perform a review write a comment saying: @ansys-reviewer-bot review |
for more information, see https://pre-commit.ci
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4142 +/- ##
==========================================
+ Coverage 79.46% 81.94% +2.47%
==========================================
Files 182 182
Lines 63852 63854 +2
==========================================
+ Hits 50739 52322 +1583
+ Misses 13113 11532 -1581 |
…3774_GDSii_Import_in_HFSS
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
…3774_GDSii_Import_in_HFSS
for more information, see https://pre-commit.ci
@amichel0205 there are still many lines not covered by test, I'd improve that. |
Fixed typo and improve clarity of commenst
Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com>
Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com>
for more information, see https://pre-commit.ci
Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com>
Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com>
…3774_GDSii_Import_in_HFSS
Update test_20_HFSS.py/test_68_import_gds_3d Add example section in documentation section
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
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
…3774_GDSii_Import_in_HFSS
work in non graphical mode
for more information, see https://pre-commit.ci
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
Create new function to import GDS file in HFSS.
It seems this function works only in Graphical Mode.
Close #3992