manifold icon indicating copy to clipboard operation
manifold copied to clipboard

Comparison against other libraries

Open pca006132 opened this issue 2 years ago • 7 comments

We can add some charts in our README to compare the performance of manifold to other libraries. We have some data comparing against OpenSCAD but it was a bit outdated (we got some performance improvement after that and did not care about tracking the data).

Some libraries to compare against:

  • OpenSCAD
  • CGAL
  • JSCAD

I can write some code to generate the OpenSCAD and JSCAD version of the same js/python script. However, I have no idea about CGAL.

pca006132 avatar Sep 24 '22 14:09 pca006132

Sounds good. OpenSCAD uses CGAL, so they aren't entirely independent. I assume this is what you're referring to as outdated (also not terribly precise). In addition we have a CGAL comparison contributed to our library.

elalish avatar Sep 25 '22 04:09 elalish

OK so is this the only blocker for release 2.0.0?

pca006132 avatar Sep 30 '22 08:09 pca006132

This feels more nice-to-have than release-blocker to me. For the release, I'd really like to publish npm and pypi packages, which means I'd like to translate and test our examples so we know those bindings are working. Would you be interested in making a modified colab based on this to test our python examples?

elalish avatar Sep 30 '22 14:09 elalish

Yes I can have a look at this.

pca006132 avatar Sep 30 '22 15:09 pca006132

Also, your comment on missing APIs in our bindings is a pretty high priority too.

elalish avatar Sep 30 '22 15:09 elalish

This feels more nice-to-have than release-blocker to me. For the release, I'd really like to publish npm and pypi packages, which means I'd like to translate and test our examples so we know those bindings are working. Would you be interested in making a modified colab based on this to test our python examples?

Do we need pyrender for preview? It seems that trimesh already provides this nicely. Here is an example.

pca006132 avatar Sep 30 '22 16:09 pca006132

Oh, nice; I didn't realize that. Sure, that sounds fine for now.

elalish avatar Sep 30 '22 17:09 elalish

probably not needed for now

pca006132 avatar Nov 28 '23 18:11 pca006132