hypar-io / Elements

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

Real mid and more #993

Open jamesbradleym opened 1 year ago

jamesbradleym commented 1 year ago

BACKGROUND:

DESCRIPTION:

TESTING:

FUTURE WORK:

REQUIRED:

COMMENTS:


This change is Reviewable

ikeough commented 1 year ago

A couple of high level comments. I'm moving this out to 2.1 for further design. Much of what is in here should be, if we choose to bring it in, part of the IBoundedCurve interface, which is the base interface for all bounded curves. Circle is not a bounded curve, so it should not have an arc length. However, Arc which is a trimmed curve with a Circle for its basis curve should have an arc length. As should EllipticalArc. We can discuss more when 2.0 is out the door.