Page MenuHome

Add Custom Falloff Curve to the Vertex Weight Proximity Modifier
ClosedPublic

Authored by Christian Friedrich (rbx775) on Nov 18 2020, 4:27 PM.

Details

Summary

The Vertex Weight Edit Modifier already got the Custom Curve so it could be added here too.

Diff Detail

Event Timeline

Christian Friedrich (rbx775) requested review of this revision.Nov 18 2020, 4:27 PM
Christian Friedrich (rbx775) created this revision.
Bastien Montagne (mont29) requested changes to this revision.Nov 20 2020, 12:52 PM
Bastien Montagne (mont29) added inline comments.
source/blender/modifiers/intern/MOD_weightvgproximity.c
259–311

Any rational for that change?

This revision now requires changes to proceed.Nov 20 2020, 12:52 PM

Thank you Bastien,

honest answer: no. :)
I think I was copying VertexWeightEdit Modifier too closely.

Should be better now.

Thanks, LGTM now

This revision is now accepted and ready to land.Nov 20 2020, 4:57 PM

Notes: While actually testing the patch before committing realized there were still tow issues with it:

  1. An assert about proximity modifier not using the 'custom curve' falloff was left in the code, should have been removed.
  2. Versioning code was missing, to add the cmap curve to existing Proximity modifier upon reading (otherwise existing files would just crash).

Remember to always use debug builds when developing, and to always also check with existing .blend files. ;)