Page MenuHome

Sculpt: Split normal radius and area radius
ClosedPublic

Authored by Pablo Dobarro (pablodp606) on Oct 4 2019, 11:28 PM.
Tags
None
Tokens
"Yellow Medal" token, awarded by 1D_Inc."Love" token, awarded by DestonXI."Love" token, awarded by RodDavis."Love" token, awarded by JulienKaspar."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

Event Timeline

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

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.

1748–1749

Same here

This revision is now accepted and ready to land.Oct 8 2019, 11:13 PM
  • Rebase
  • Fix artifacts with area radius and pressure size

I needed to rewrite all the sculpt.c part of this patch to rebase it on top of the changes to the scheduler and the tasks refactor, but it does the same thing.

  • Fix missing property
  • Enable area radius in fill brush