Rename Polygon -> Path and remove unused functions

voneiden commented 1 year ago

Rationale for renaming is

a) Path is shorter b) Path can represent OpenPath and ClosedPath better than the term Polygon, IMO

codecov-commenter commented 1 year ago

Codecov Report

Merging #24 (e3d7b83) into unstable (8e059c7) will increase coverage by 0.63%. The diff coverage is 77.58%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@             Coverage Diff              @@
##           unstable      #24      +/-   ##
============================================
+ Coverage     71.29%   71.93%   +0.63%     
============================================
  Files            33       33              
  Lines          2191     2159      -32     
============================================
- Hits           1562     1553       -9     
+ Misses          629      606      -23

Impacted Files	Coverage Δ
src/cq_cam/utils/utils.py	`85.18% <ø> (+7.09%)`	:arrow_up:
src/cq_cam/operations/pocket.py	`17.70% <44.44%> (ø)`
src/cq_cam/routers.py	`52.94% <75.00%> (ø)`
src/cq_cam/utils/geometry_op.py	`74.64% <80.55%> (+0.36%)`	:arrow_up:
src/cq_cam/operations/strategy.py	`63.88% <100.00%> (+0.25%)`	:arrow_up:
src/cq_cam/utils/tests/test_polyface.py	`100.00% <100.00%> (ø)`
src/cq_cam/utils/tests/test_segment_math.py	`100.00% <100.00%> (ø)`

sonarcloud[bot] commented 1 year ago

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

77.6% Coverage
0.0% Duplication

voneiden / cq-cam

Rename Polygon -> Path and remove unused functions #24

Codecov Report