-
Notifications
You must be signed in to change notification settings - Fork 40
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
extra geometries for 2d material #1644
Conversation
92b5959
to
780a925
Compare
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.
Looks good! Thanks @dmarek-flex. One minor question
780a925
to
70055dc
Compare
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.
Nice work! All look good.
70055dc
to
54a135d
Compare
54a135d
to
4b5c717
Compare
I made the modifications that @tylerflex suggested in the coaxial ports PR |
7894b82
to
aeb2594
Compare
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.
Looks much better I think! Thanks @dmarek-flex. Couple minor comments
added validation tests for structures with 2d materials and methods for updating bounds of certain geometries
aeb2594
to
d78268a
Compare
Thanks @tylerflex for finding those lingering issues. I'll merge this when all tests pass. |
Added the ability for Transformed/GeometryGroup/ClipOperation geometries to be composed of 2D materials.
Solves issue 576