hypar-io / Elements

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

Add Intersect method for Surfaces #110

Open radumg opened 5 years ago

radumg commented 5 years ago

Further to the improvements requested in #56 , at @designtechIO we've identified we need the following additional geometric types/operation :

Surface.Intersect - used to intersect two surfaces

Further notes on this from @ikeough that we (@radumg @markthorley) agree with :

All surfaces in Hypar are currently planar, so intersection of surfaces will result in a line segment bounded by some edge curves of the surface. We do not have this currently, but we have all the underlying intersection methods

arif-hanif commented 5 years ago

this method should also have a overload to align surfaces in 3d space if the two surfaces are not coplanar

radumg commented 5 years ago

I'd argue against that @arif-hanif as that sounds like a side-effect, think it would be better to have alignment as separate functionality where it makes sense

arif-hanif commented 5 years ago

makes sense