Skip to content
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

Split and repeat on boundary method #352

Draft
wants to merge 22 commits into
base: main
Choose a base branch
from

Conversation

JackB-Ansys
Copy link
Collaborator

Adding a new region method to split region 1 when it overhangs region 2 and rotate the overhanging portion of region 1. Devised to account for regions that overhang the rotor or stator boundary to simplify the process of mirroring or repeating drawing of a region to the other side of the rotor or stator.

(Illustration to be added)

Added class method for Region boundary_split()
…d_repeat_on_boundary_method

# Conflicts:
#	src/ansys/motorcad/core/geometry.py
#	tests/test_geometry.py
@JackB-Ansys JackB-Ansys added the enhancement New features or code improvements label Jul 31, 2024
@JackB-Ansys JackB-Ansys self-assigned this Jul 31, 2024
Copy link

codecov bot commented Jul 31, 2024

Codecov Report

Attention: Patch coverage is 40.44944% with 53 lines in your changes missing coverage. Please review.

Project coverage is 82.69%. Comparing base (3dad907) to head (e81edab).
Report is 10 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #352      +/-   ##
==========================================
- Coverage   86.65%   82.69%   -3.96%     
==========================================
  Files          20       21       +1     
  Lines        2098     2277     +179     
==========================================
+ Hits         1818     1883      +65     
- Misses        280      394     +114     

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New features or code improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants