pca006132

another thing is that manifold is not really optimized for memory usage.

Interesting, maybe someone can use gdb to see what is causing it to crash?

I am not entirely sure why it crashes for you on windows, but it is definitely running out of memory for me on Linux. And I think in general it...

In addition, the cache will make manifold unable to flatten the tree internally because it thinks that the node may be reused in a different part. This is why I...

Looking more closely at this, I think this will run OOM even if we disable the cache and remove don't run it in multiple threads, but ideally we should be...

btw, thinking about it, we should also perform toPolySet even when the cache is hit, because the result can be a `ManifoldGeometry`. Will fix when I actually wake up... (suddenly...

I was thinking if it would be better to just convert to PolySet when caching ManifoldGeometry, but one problem is that this triggers evaluation of the underlying Manifold object, which...

That said, I think we will still have backend specific geometry type vs PolySet, as primitives are returned as PolySets? Agree this is the more maintainable way though.

@kintel so what should we do with this PR? remove the cache related stuff to properly fix it? may require some refactoring if we want to do it in the...

btw, wonder if this will make more sense: ```diff diff --git a/src/geometry/GeometryEvaluator.cc b/src/geometry/GeometryEvaluator.cc index a721f92fb..f78ab1126 100644 --- a/src/geometry/GeometryEvaluator.cc +++ b/src/geometry/GeometryEvaluator.cc @@ -80,9 +80,12 @@ std::shared_ptr GeometryEvaluator::evaluateGeometry(const Abstra // We cannot...