Page MenuHome

Fix T72251: Add rotate brush as constrained by radius for automasking
ClosedPublic

Authored by Pablo Dobarro (pablodp606) on Dec 7 2019, 8:00 PM.

Details

Summary

This brush should be added to the set of brushes where we know which
vertices are going to be affected by the brush when starting the stroke.
This way we can limit the automasking only to those vertices instead of
flood filling the whole mesh from the active vertex.

All brushes that are not in this set will automask by flood filling the
mesh when starting the stroke. To improve this and make it work as most
users expect, we need a fast way to calculate topological distances on
high poly meshes.

Diff Detail

Repository
rB Blender

Event Timeline

The bug report also mentions the inflate brush, so not sure if this is a complete fix or if that was intentionally left out.

Might also be missing brushes like elastic deform? Not sure.

@Brecht Van Lommel (brecht) Inflate works with stroke spacing (it does not have an anchored location with a fixed radius), so it is like any other brush.
Elastic deform does, but the current version needs to modify all vertices regarding of the radius. If we stop propagating the automasking values at the boundary of the brush radius it will produce artifacts.

This revision is now accepted and ready to land.Tue, Jan 7, 11:31 AM