Add support for 2D and 3D Wire fillet

[dov]

Add support for 2D and 3D Wire fillet through OCCT ChFi2d_FilletAPI

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.54%. Comparing base (76fbff7) to head (d20379a).
Report is 2 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1549      +/-   ##
==========================================
+ Coverage   94.50%   94.54%   +0.03%     
==========================================
  Files          28       28              
  Lines        5804     5846      +42     
  Branches     1076     1165      +89     
==========================================
+ Hits         5485     5527      +42     
  Misses        193      193              
  Partials      126      126              

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

[adam-urbanczyk]

Hey, thanks, but could you not force push

[dov]

@adam-urbanczyk
Regarding force push. Sorry, I was (ab-)using it in order to get the CI support. I'll make sure in the future to clean all the CI errors before creating pull requests.

Regarding the PR itself, I added a vertices parameter, but in terms of a list of vertex indices. I was wondering whether it might be better to give the coordinates. What do you think?

[adam-urbanczyk]

Every push will trigger CI, am I missing something?

[adam-urbanczyk]

More work will be needed on this. I'll try to add startVertex() and endVertex() methods.

[dov]

🥳

[adam-urbanczyk]

FYI: in the end I refactored Wire.Vertices() so that the rv is ordered.

[jmwright]

Thanks @dov there is just the one comment suggestion from me.

[dov]

Thanks @dov there is just the one comment suggestion from me.

Thanks @jmwright . I fixed it and also removed another typo in the comment.

[adam-urbanczyk]

Thanks @dov !