Page MenuHome

Remove Carve boolean
ClosedPublic

Authored by Sergey Sharybin (sergey) on Feb 7 2018, 2:47 PM.

Details

Summary

We've got quite comprehensive BMesh based implementation, which is way easier
for maintenance than abandoned Carve library.

After all the time BMesh implementation was working on the same level of
limitations about manifold meshes and touching edges than Carve. Is better
to focus on maintaining one boolean implementation now.

Diff Detail

Repository
rB Blender
Branch
carve_removal
Build Status
Buildable 1182
Build 1182: arc lint + arc unit

Event Timeline

Tested, LGTM. We can remove BooleanSolver too.

This revision is now accepted and ready to land.Feb 7 2018, 3:02 PM

Remove unused enum

This revision was automatically updated to reflect the committed changes.
Dmitry (Dima) reopened this revision.EditedFeb 21 2019, 7:08 AM
Dmitry (Dima) added a subscriber: Dmitry (Dima).

Bmesh is not good enough to replace Carve everywhere. I've attached an example.

This revision is now accepted and ready to land.Feb 21 2019, 7:08 AM

Re-opening this revision isn't helpful, closing.

BMesh boolean related TODO's are here: T47030

I was wrong. My example contains non-manifold objects so it's not suitable for bmesh/carve comparison, as I read there https://developer.blender.org/T59357. I've made manifold objects from that non-manifolds and it works. Previously I thought there is no any way to make that kind of non-destructive cuts in Blender newer than 2.79b.