-
Notifications
You must be signed in to change notification settings - Fork 66
Singularity correction at metallic edges #2498
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
Conversation
c57b1a0
to
e0ca6b6
Compare
Edge correction is now only applied at PEC edges. |
Code Coverage Summary
Diff against develop
Results for commit: e0ca6b6 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
Changed Files Coverage
Results for commit: e0ca6b6 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
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.
Frontend looks good
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.
3 files reviewed, 1 comment
Edit PR Review Bot Settings | Greptile
e0ca6b6
to
b8e9410
Compare
Diff CoverageDiff: origin/develop...HEAD, staged and unstaged changes
Summary
|
b8e9410
to
81b4de0
Compare
@dmarek-flex For back-compatibility, we should set the default edge singularity correction to |
ec361d4
to
5bbbd64
Compare
singularity correction for lossy metal is now also enabled. |
5bbbd64
to
a77d998
Compare
a77d998
to
337a213
Compare
Greptile Summary
Added field singularity correction capabilities for Perfect Electric Conductor (PEC) edges in electromagnetic simulations, improving accuracy at metallic boundaries.
edge_singularity_correction
parameter intidy3d/components/subpixel_spec.py
for PECConformal class with default=Truetests/test_components/test_simulation.py
for edge correction behavior