Page MenuHome

Fix T82072: Blender crash/hangs with "voxel size" low values in remesh modifier
AbandonedPublic

Authored by Manuel Castilla (manzanilla) on Oct 26 2020, 12:58 AM.

Details

Summary

Solution:
Do not allow "voxel size" values lower than "0.01" in UI. Only if user introduce it manually. For most users blender would hang for a long time or even hang PC.

Diff Detail

Repository
rB Blender
Branch
T82072 (branched from master)
Build Status
Buildable 10912
Build 10912: arc lint + arc unit

Event Timeline

Manuel Castilla (manzanilla) requested review of this revision.Oct 26 2020, 12:58 AM
Manuel Castilla (manzanilla) retitled this revision from Fix T82072 Solution: Do not allow "voxel size" values lower than "0.01". Beyond that is too CPU intensive. to Fix T82072: Blender crash/hangs with "voxel size" low values in remesh modifier.Oct 26 2020, 1:01 AM
Manuel Castilla (manzanilla) edited the summary of this revision. (Show Details)
Hans Goudey (HooglyBoogly) requested changes to this revision.Oct 26 2020, 1:14 AM

Doesn't whether it is too CPU intensive completely depends on the size of the object and the CPU in question? Changing the range could improve the experience in many cases, but it will also cause problems for other people.

I would recommend only changing the ui_range so people can still use the modifier for small objects.

This revision now requires changes to proceed.Oct 26 2020, 1:14 AM

yes, it's more reasonable what you propose.
To be precise the CPU is Intel(R) Core(TM) i7-6700K CPU @ 4.00GHz

With 0.001 it hangs during 20-30 seconds.
With 0.0001 pc hangs and have to reboot manually.

I just tested with the default cube

So I really think for most users is bad experience. What you said is probably the best solution.

T82072 is a duplicate of T77868

This patch should be considered along side patch D9074 since that one also attempts to mitigate the issue.

Thanks for pointing out. +1 for D9074: Add Non Linear Sliders , that's the proper way of trying to fix it.

In that case maybe it makes more sense to close this patch and focus efforts on the non-linear sliders.

I've had to force close Blender because of this too, it's quite annoying for sure.