Page MenuHome

Sculpt: Split normal radius and area radius
AcceptedPublic

Authored by Pablo Dobarro (pablodp606) on Fri, Oct 4, 11:28 PM.
Tags
None
Subscribers
None
Tokens
"Love" token, awarded by MetinSeven."Like" token, awarded by knightknight."Love" token, awarded by michaelknubben."100" token, awarded by Frozen_Death_Knight."Love" token, awarded by jfmatheu."Love" token, awarded by Zino."Love" token, awarded by Brandon777."Love" token, awarded by monio."Like" token, awarded by Loner.

Details

Summary

This enables an extra layer of control in the sculpt brushes.
For now it is enabled only in Scrape, but it should work in all brushes (like normal radius). I think we can safely enable this in 2.81 if we keep it this way.
You can tweak in this property in the scrape brush to achieve a much better behavior when working on curve surfaces and control how much volume you want to trim. In most cases, it also fixes the bug where the brush keeps trimming in the same area without disabling accumulate. I can probably fix some other artifacts in other brushes by tweaking this value.

Diff Detail

Repository
rB Blender
Branch
sculpt-area-radius-factor (branched from master)
Build Status
Buildable 5249
Build 5249: arc lint + arc unit

Event Timeline

Jeroen Bakker (jbakker) added inline comments.
source/blender/editors/sculpt_paint/sculpt.c
1645

Not related to this change but we should be consistent. Or use ARRAY_SIZE or a fixed number. In this case a fixed number would be more wise as it will not work when used with a different input array size.

1756

Same here

This revision is now accepted and ready to land.Tue, Oct 8, 11:13 PM