Page MenuHome

Fix Solidify Complex Dissolve Crash (fixes T74195)
ClosedPublic

Authored by Henrik Dick (weasel) on Mar 22 2020, 7:41 PM.

Details

Diff Detail

Repository
rB Blender

Event Timeline

Crash is fixed indeed, however now with high resolutions (15 or more), the letters shape start to get badly affected, which does not happen with the 'simple' method... No idea whether this is a limitation of the complex method? It seems to be merging too much when input geometry density gets high?

The merging behaviour is intended to handle degenerate cases. If your mesh is really tiny this will occur. I could expose the Merge Threshold, but changing it could lead to serious precision problems everywhere in the code. It is recommended to scale the mesh up in editmode and scale it down in object mode. That will fix this problem without exposing the Threshold.
There is no mention of the handling of degenerate cases in the manual sadly because I totally forgot about that feature whilst updating the docs of the old solidify. I will add a mention in the docs when I find myself in the mood to do some documentation again.

I would not consider file in T74195 as unreasonably small, it's 0.05 units wide, which means that at somewhat high level of subdivision you get edges in the 0.0001 area, I would expect this to be working still. So yes, think exposing the merge threshold would be a good thing (also because we do it in lots of other places in Blender). In any case, we are working with floating numbers here, so as long as the size of the details is not to small compared to the whole model size (or rather, offset from origin), there should be no issue. If you really do not want to expose the threshold to users, then it should at least be computed e.g. relative to the farthest away vertex from origin.

Other issue with this merge is that it seems to be weirdly cumulative, ending merging whole big chunks of the geometry, which should not happen imho.

I should probably create another report for that issue though, as it is not related to that fix. patch itself LGTM, will commit, thanks.

This revision was not accepted when it landed; it landed in state Needs Review.Mar 23 2020, 3:26 PM
This revision was automatically updated to reflect the committed changes.