Transfer weights function does not account for mirror #38265

New Issue

Zauber Paracelsus · 2014-01-17T17:37:32+01:00

Zauber Paracelsus commented

2014-01-17 17:37:32 +01:00

System Information
64bit Linux Mint 16
Linux Kernel 3.11.0
nVidia GTX 650 Ti, driver 319.32

Blender Version
Broken: 2.69.0 r60991

Short description of error
When in weight painting mode and transferring the weights from an object that uses the mirror modifier to one that doesn't, the

Exact steps for others to reproduce the error

Model an object, and set it to use the mirror modifier
Model another object of similar shape, set to use the mirror modifier as well.
Apply the mirror modifier on the second object.
Create an armature, and parent the first object to it, with automatic weights.
Parent the second object to the armature, with empty vertex groups
Deselect all, select the first object, and then select the second object. Make sure the first object is still selected.
Go into weight painting mode, and click the "Transfer Weights" option in the tool shelf.

**System Information** 64bit Linux Mint 16 Linux Kernel 3.11.0 nVidia GTX 650 Ti, driver 319.32 **Blender Version** Broken: 2.69.0 r60991 **Short description of error** When in weight painting mode and transferring the weights from an object that uses the mirror modifier to one that doesn't, the **Exact steps for others to reproduce the error** 1) Model an object, and set it to use the mirror modifier 2) Model another object of similar shape, set to use the mirror modifier as well. 3) Apply the mirror modifier on the second object. 4) Create an armature, and parent the first object to it, with automatic weights. 5) Parent the second object to the armature, with empty vertex groups 6) Deselect all, select the first object, and then select the second object. Make sure the first object is still selected. 7) Go into weight painting mode, and click the "Transfer Weights" option in the tool shelf.

Zauber Paracelsus commented

2014-01-17 17:37:32 +01:00

Changed status to: 'Open'

Zauber Paracelsus commented

2014-01-17 17:37:32 +01:00

Added subscriber: @ZauberParacelsus

Brecht Van Lommel commented

2014-01-20 19:37:37 +01:00

Added subscribers: @cyborgmuppet, @ideasman42, @brecht

Brecht Van Lommel commented

2014-01-20 19:37:37 +01:00

The transfer weight function doesn't take modifiers of the source mesh into account. This is fairly easy to fix, see this patch transfer_weight_from_dm.diff.

But now I'm not so sure if this should be considered a bug. In this case it's useful, but if for example one of the meshes has a subsurf modifier, or both meshes have it, then this patch is not going to work well. Should there be some option to take modifiers into account, or should the user just (temporarily) apply the modifiers when transferring weights?

@cyborgmuppet and @ideasman42, what do you think?

The transfer weight function doesn't take modifiers of the source mesh into account. This is fairly easy to fix, see this patch [transfer_weight_from_dm.diff](https://archive.blender.org/developer/F71044/transfer_weight_from_dm.diff). But now I'm not so sure if this should be considered a bug. In this case it's useful, but if for example one of the meshes has a subsurf modifier, or both meshes have it, then this patch is not going to work well. Should there be some option to take modifiers into account, or should the user just (temporarily) apply the modifiers when transferring weights? @cyborgmuppet and @ideasman42, what do you think?

blender-admin commented

2014-01-21 15:36:00 +01:00

This issue was referenced by blender/blender-addons-contrib@386d3a9cfb

This issue was referenced by blender/blender-addons-contrib@386d3a9cfbed27b4a9ef5e478d50f38a068f03fe

blender-admin commented

2014-01-21 15:36:00 +01:00

This issue was referenced by 386d3a9cfb

This issue was referenced by 386d3a9cfbed27b4a9ef5e478d50f38a068f03fe

Brecht Van Lommel commented

2014-01-21 15:42:08 +01:00

Committed it now anyway, since this would actually crash because it was using the vertex coordinates array from the modified mesh and vertex weights array from the unmodified mesh. As far as I can tell the intention was always to use the modified mesh, so this fix doesn't really break any working behavior.

Brecht Van Lommel commented

2014-01-21 15:42:33 +01:00

Changed status from 'Open' to: 'Resolved'

Brecht Van Lommel closed this issue

2014-01-21 15:42:33 +01:00

Brecht Van Lommel commented

2014-01-21 15:42:33 +01:00

Closed by commit 386d3a9cfb.

Closed by commit 386d3a9cfb.

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

Transfer weights function does not account for mirror #38265