This implements some basic improvements to the Snake Hook brush. It now supports a spherical/3D brush and different falloff modes.
Details
Details
- Reviewers
Hans Goudey (HooglyBoogly) - Maniphest Tasks
- T96448: Snake Hook - curve sculpting
- Commits
- rBfced604acfc9: Curves: improve Snake Hook brush
Diff Detail
Diff Detail
- Repository
- rB Blender
- Branch
- snake-hook-brush
- Build Status
Buildable 21311 Build 21311: arc lint + arc unit
Event Timeline
Comment Actions
I notice that sometimes I pick up extra curve tips that weren't in the initial brush radius. Is that on purpose? I think ideally it wouldn't do that or it would be a choice. It only happens sometimes-- maybe when the tips are at the same depth as the starting point?
Other than that, the patch looks good to me.
| source/blender/editors/sculpt_paint/curves_sculpt_snake_hook.cc | ||
|---|---|---|
| 235–236 | I wonder if this function would be simpler if you sliced the positions argument as part of calling it, and didn't pass IndexRange points | |
Comment Actions
I notice that sometimes I pick up extra curve tips that weren't in the initial brush radius. Is that on purpose? I think ideally it wouldn't do that or it would be a choice. It only happens sometimes-- maybe when the tips are at the same depth as the starting point?
I'd be fine with changing that, but right now the behavior seems to match the snake hook brush for meshes. Having an option to switch between both behaviors is fine for me as well, but I think that should then be done as a separate patch.