Skip to content

Added a method to find the face closest to bounding box (in case thee is no face touching the bounding box). #1249

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

Merged
merged 3 commits into from
Jun 7, 2022

Conversation

maxcapodi78
Copy link
Collaborator

Also added a method in Hfss3dLayout to filder objects by layer and by nets.

…e is no face touching the bounding box).

Also added a method in Hfss3dLayout to filder objects by layer and by nets.
@maxcapodi78 maxcapodi78 requested a review from svandenb-dev June 7, 2022 07:57
maxcapodi78 added 2 commits June 7, 2022 10:18
…e is no face touching the bounding box).

Also added a method in Hfss3dLayout to filder objects by layer and by nets.
…e is no face touching the bounding box).

Also added a method in Hfss3dLayout to filder objects by layer and by nets.
@codecov
Copy link

codecov bot commented Jun 7, 2022

Codecov Report

Merging #1249 (c04f0d8) into main (dcb1b47) will increase coverage by 0.05%.
The diff coverage is 94.80%.

@@            Coverage Diff             @@
##             main    #1249      +/-   ##
==========================================
+ Coverage   77.58%   77.64%   +0.05%     
==========================================
  Files          96       96              
  Lines       37782    37864      +82     
==========================================
+ Hits        29315    29399      +84     
+ Misses       8467     8465       -2     

@maxcapodi78 maxcapodi78 merged commit bd4b47d into main Jun 7, 2022
@maxcapodi78 maxcapodi78 deleted the hfss3dlayout_modeler_methods branch June 7, 2022 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants