Maniphest T61119

Sculpt mode ignores shape key when the object has a Mask modifier.
Closed, Resolved

Assigned To
Sergey Sharybin (sergey)
Authored By
Demeter Dzadik (Mets)
Feb 2 2019, 12:24 AM
Tags
  • BF Blender
Subscribers
Brecht Van Lommel (brecht)
Demeter Dzadik (Mets)
Sebastian Parborg (zeddb)

Description

Blender Version
Broken: 2.80, a1ae04d15a9f

Short description of error
If your object has Mask modifier, entering sculpt mode on a shape key will ignore any existing data in that shape key. It will be as if you are sculpting a brand new shape key. When you leave sculpt mode, the "fake" shape key you've been sculpting will be added onto the actual shape key that you were meant to be sculpting.

Also note that the vertex group that is specified for the Mask modifier must have some verts assigned to it to produce the bug, even if the assigned weights are 0. The bug doesn't happen if no vertex group is selected.

Exact steps for others to reproduce the error

  • Download attached blend
  • Note how the currently active shape key turns the cube into a sphere.
  • Enter sculpt mode
  • The mesh turns back into a cube (as if the shape key didn't exist)

Workaround
Disabling the mask modifier while sculpting makes the problem go away.

I noticed this warning on the modifier when entering sculpt mode


But note that this warning existed in 2.7 as well, but in 2.7 Mask modifiers work perfectly fine when sculpting a shape key, so I find this warning odd in any case.

Related Objects

Mentioned Here
rBa1ae04d15a9f: Versioning: add renaming utility function
T60389: Object preview differences in Sculpt Mode

Event Timeline

Demeter Dzadik (Mets) created this task.Feb 2 2019, 12:24 AM
Sebastian Parborg (zeddb) assigned this task to Sergey Sharybin (sergey).Feb 4 2019, 11:37 AM
Sebastian Parborg (zeddb) lowered the priority of this task from 90 to 50.
Sebastian Parborg (zeddb) added a subscriber: Sebastian Parborg (zeddb).

I'm guessing this is related to T60389 (but as that is more of an collection of bugs, I'm unsure if I should merge this in or not).

Demeter Dzadik (Mets) updated the task description.Mar 29 2019, 5:24 PM
Demeter Dzadik (Mets) updated the task description.
Brecht Van Lommel (brecht) changed the task status from Unknown Status to Resolved.May 29 2019, 3:00 PM
Brecht Van Lommel (brecht) added a subscriber: Brecht Van Lommel (brecht).

Confirmed in rBa1ae04d15a9f: Versioning: add renaming utility function, but appears to be fixed now.