Page MenuHome

Fix T7095: Add option to fill holes before running the voxel remesher
Needs RevisionPublic

Authored by Pablo Dobarro (pablodp606) on Oct 21 2019, 6:01 PM.
Tags
None
Tokens
"Love" token, awarded by ruthwikrao."100" token, awarded by Frozen_Death_Knight."Love" token, awarded by Brandon777."Love" token, awarded by amonpaike.

Details

Summary

You can do this same operation by going to edit mode and running the fill holes operator. I'm adding this to the remesher to make the workflow easier. In the future, we may have tools that produce holes in the mesh.
It should be disabled by default. It is an extra processing step that is not useful in the majority of cases.

Diff Detail

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

Event Timeline

Jeroen Bakker (jbakker) requested changes to this revision.Oct 30 2019, 8:33 AM
Jeroen Bakker (jbakker) added inline comments.
source/blender/editors/object/object_remesh.c
128

Best to not copy, but add a flag if this if input_mesh needs to be freed.
This saves a lot of memory allocations and copying

This revision now requires changes to proceed.Oct 30 2019, 8:33 AM
source/blender/editors/object/object_remesh.c
128

But if I don't copy the mesh I will be modifying the original mesh directly with fill_holes and OpenVDB can fail before copying the remesher result to the original mesh.