Skip to content

Feat: change Segment to accept analytic functions#65

Merged
carlos-adir merged 2 commits into
mainfrom
dev/analytic_segment
Aug 29, 2025
Merged

Feat: change Segment to accept analytic functions#65
carlos-adir merged 2 commits into
mainfrom
dev/analytic_segment

Conversation

@carlos-adir

Copy link
Copy Markdown
Contributor

Currently the Segment accepts the control points because it's a Bezier curve.

To further implementation of circles, trigonometric analytic functions will be used to allows exact circles.

Hence, Segment now receives as parameters xfunc and yfunc, the analytic functions for the segment

@carlos-adir carlos-adir self-assigned this Aug 29, 2025
@carlos-adir carlos-adir added the enhancement New feature or request label Aug 29, 2025
@carlos-adir carlos-adir linked an issue Aug 29, 2025 that may be closed by this pull request
@carlos-adir carlos-adir marked this pull request as ready for review August 29, 2025 21:45
@codecov

codecov Bot commented Aug 29, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.55%. Comparing base (6539577) to head (3cce7ca).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #65      +/-   ##
==========================================
- Coverage   98.63%   98.55%   -0.09%     
==========================================
  Files          26       26              
  Lines        2496     2489       -7     
==========================================
- Hits         2462     2453       -9     
- Misses         34       36       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@carlos-adir carlos-adir reopened this Aug 29, 2025
@carlos-adir carlos-adir merged commit 262f930 into main Aug 29, 2025
11 of 12 checks passed
@carlos-adir carlos-adir deleted the dev/analytic_segment branch October 26, 2025 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Expand Segment capabilities to more than Bezier Curve

1 participant