-
Notifications
You must be signed in to change notification settings - Fork 162
Find the largest area rectangle in a polygon. #854
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
Made vector function compatible with 2D vectors.
Codecov Report
@@ Coverage Diff @@
## main #854 +/- ##
==========================================
+ Coverage 79.81% 80.21% +0.40%
==========================================
Files 128 128
Lines 36493 36949 +456
==========================================
+ Hits 29126 29640 +514
+ Misses 7367 7309 -58 |
increased unit tests.
get_mid_point works for any vector length.
Co-authored-by: Maxime Rey <[email protected]>
… into Enhancement/configs
# Conflicts: # pyaedt/edb.py
…ctangle_inside_polygon
# Conflicts: # _unittest/conftest.py # pyaedt/generic/general_methods.py
Co-authored-by: Maxime Rey <[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.
Overall it's a great feature. only minor comments
# Conflicts: # _unittest/test_00_EDB.py # _unittest/test_04_GeometryOperators.py # pyaedt/modeler/GeometryOperators.py
# Conflicts: # _unittest/test_00_EDB.py
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
Find the largest area rectangle in a polygon with arbitrary orientation.
Added new functions to GeometryOperators.
Made vector function compatible with 2D vectors.