hypar-io / Elements

The smallest useful BIM.
https://www.hypar.io
MIT License
354 stars 74 forks source link

Arc by three points and better fillets. #962

Closed ikeough closed 1 year ago

ikeough commented 1 year ago

This PR requires #957.

BACKGROUND: Building on the work of #957 and #959, I'd like to be able to build arcs by three points. IFCIndexedPolyCurve requires this and the ability to build an arc by three points is generally useful.

DESCRIPTION:

TESTING: Is there a scenario where we shouldn't be able to create an arc by three points? Here's the ones I've captured with tests.

FUTURE WORK:

REQUIRED:

image

This change is Reviewable