-
Notifications
You must be signed in to change notification settings - Fork 161
FEAT: Uncover face #6122
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
FEAT: Uncover face #6122
Conversation
self.aedtapp.modeler[xx] will return the FacePrimitive or EdgePrimitive depending to which the id it belongs
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 #6122 +/- ##
=======================================
Coverage 85.24% 85.25%
=======================================
Files 167 167
Lines 63637 63657 +20
=======================================
+ Hits 54247 54269 +22
+ Misses 9390 9388 -2 🚀 New features to boost your workflow:
|
@jvela018 Is it not possible to uncover multiple faces in one single call? Please test it, because if the API allows multi-face selection, the PyAEDT method should also allow it. Thanks! |
@Samuelopez-ansys I added multi face selection. Is this implementation ok? Thanks |
Co-authored-by: Samuel Lopez <[email protected]>
Co-authored-by: Samuel Lopez <[email protected]>
Co-authored-by: Samuel Lopez <[email protected]>
Co-authored-by: Samuel Lopez <[email protected]>
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
Description
Creation of a method uncover_face in the modeler class.
Issue linked
#6100