Mirror modifier does not Custom Split Normal Data #63840

New Issue

Jacob Max Fober · 2019-04-23T22:57:07+02:00

Jacob Max Fober commented

2019-04-23 22:57:07 +02:00

System Information
Operating system: Windows-7-6.1.7601-SP1 64 Bits
Graphics card: GeForce GT 1030/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 385.12

Blender Version
Broken: version: 2.80 (sub 58), branch: master, commit date: 2019-04-23 12:31, hash: 5a144c797a

Short description of error
When using Custom Split Normal Data (or just custom vertex normal) with mirror modifier, normal vectors are not mirrored properly.

Exact steps for others to reproduce the error

In default Cube, Object Data, Normals, enable Auto Smooth
Move base cube in edit mode off the origin center
Select all and go to Mesh, Normals, Set From Faces
Select top face and repeat Set From Faces
Add mirror modifier to the object

Results are different, not mirrored normals.

[Based on the attached .blend file (as simple as possible)]
Mirror_Normal_Bug.blend

**System Information** Operating system: Windows-7-6.1.7601-SP1 64 Bits Graphics card: GeForce GT 1030/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 385.12 **Blender Version** Broken: version: 2.80 (sub 58), branch: master, commit date: 2019-04-23 12:31, hash: `5a144c797a` **Short description of error** When using Custom Split Normal Data (or just custom vertex normal) with mirror modifier, normal vectors are not mirrored properly. ![image.png](https://archive.blender.org/developer/F6970424/image.png) **Exact steps for others to reproduce the error** 1. In default Cube, Object Data, Normals, enable Auto Smooth 2. Move base cube in edit mode off the origin center 3. Select all and go to Mesh, Normals, Set From Faces 4. Select top face and repeat Set From Faces 5. Add mirror modifier to the object Results are different, not mirrored normals. [Based on the attached .blend file (as simple as possible)] [Mirror_Normal_Bug.blend](https://archive.blender.org/developer/F6970429/Mirror_Normal_Bug.blend)

Jacob Max Fober commented

2019-04-23 22:57:07 +02:00

Added subscriber: @Fubax-1

Jacob Max Fober changed title from ~~Mirror modifier messes up Custom Split Normal Data~~ to Mirror modifier does not Custom Split Normal Data

2019-04-23 23:01:51 +02:00

Philipp Oeser commented

2019-04-24 10:26:09 +02:00

Added subscribers: @mont29, @lichtwerk

Philipp Oeser commented

2019-04-24 10:26:09 +02:00

quoting @mont29 here:

In #53763#480326, @mont29 wrote:
Thanks for the report, but there is nothing new here, custom normals have never been supported with topology-editing modifiers or operations (like mirror one). Iirc, 2017 GSoC tackling editing of custom normals also added their support to mirror modifier, not sure when we'll have time to merge it in master though :|

But in general, you should never modify topology of your mesh once you have added custom normals, we might end up supporting a limited set of cases (like the mirror modifier), but this is not a good idea anyway. Custom normals are designed to support alterations of the shape (deformations), but absolutely not of the topology, this is not possible. So this is a known limitation/low priority TODO at best. ;)

So will close this as duplicate of #53763

quoting @mont29 here: > In #53763#480326, @mont29 wrote: > Thanks for the report, but there is nothing new here, custom normals have never been supported with topology-editing modifiers or operations (like mirror one). Iirc, 2017 GSoC tackling editing of custom normals also added their support to mirror modifier, not sure when we'll have time to merge it in master though :| > > But in general, you should never modify topology of your mesh once you have added custom normals, we might end up supporting a limited set of cases (like the mirror modifier), but this is not a good idea anyway. Custom normals are designed to support alterations of the shape (deformations), but absolutely not of the topology, this is not possible. So this is a known limitation/low priority TODO at best. ;) So will close this as duplicate of #53763

Philipp Oeser commented

2019-04-24 10:26:19 +02:00

Closed as duplicate of #53763

Philipp Oeser closed this issue

2019-04-24 10:26:19 +02:00

Francesco Sorrentino commented

2019-07-09 07:13:52 +02:00

Added subscriber: @Zeirus

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

Mirror modifier does not Custom Split Normal Data #63840