hypar-io / Elements

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

CellComplex seems to ignore tolerance? #1069

Open secretlyagoblin opened 9 months ago

secretlyagoblin commented 9 months ago

Describe the bug I've been experimenting with Hex grids and CellComplexs, which means square root of 3, which means floating point error. To remedy this, I set the CellComplex tolerance to a high number, but this seems to not work as I would expect.

To Reproduce Steps to reproduce the behavior: https://github.com/hypar-io/Elements/compare/master...secretlyagoblin:Elements:bug/cell-toerance has a test set up demonstrating the behaviour

Expected behavior I would expect any vertices below the target tolerance to be merged on addition to the CellComplex

Screenshots If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):