-
Notifications
You must be signed in to change notification settings - Fork 1
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
Fix percentage distance methods #339
Conversation
…by update to get_coordinate_from_distance methods.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #339 +/- ##
==========================================
+ Coverage 87.27% 87.47% +0.20%
==========================================
Files 20 20
Lines 2192 2212 +20
==========================================
+ Hits 1913 1935 +22
+ Misses 279 277 -2 |
@JackB-Ansys You will need to improve the existing tests to cover all the new lines of code |
Needs merge from main |
…entage_distance_methods # Conflicts: # src/ansys/motorcad/core/geometry.py # tests/test_geometry.py
… tests for new optional arguments)
…ts for warnings when multiple arguments are given)
…t for arc drawn with negative radius)
…_from_percentage_distance arc method
…fraction is provided
…entage_distance_methods
…entage_distance_methods
…entage_distance_methods
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.
A few documentation lines where line and arc need to be swapped.
Co-authored-by: James Packer <[email protected]>
Co-authored-by: James Packer <[email protected]>
Co-authored-by: James Packer <[email protected]>
Co-authored-by: James Packer <[email protected]>
Co-authored-by: James Packer <[email protected]>
Deprecate get_coordinate_from_percentage_distance methods because these actually use fractional distances. Replace this functionality by adding optional arguments for get_coordinate_from_distance, where user can specify a percentage or fraction instead of specifying an absolute distance.