Triangulate modifier breaks custom normals #61942

New Issue

Nick Sawyer · 2019-02-25T18:11:09+01:00

Nick Sawyer commented

2019-02-25 18:11:09 +01:00

Operating system: Windows 10 Pro
Graphics card: Nvidia Geforce 1070

blender-2.80.0-893fa598319e-windows64

Adding a triangulate modifer breaks any custom vertex normals, such as those created by the weighted normal modifier. It's typical in game art to add a non-destructive triangulate before exporting to a game engine or texture baking app. Weighted normals behave differently on a triangulated mesh, so it's not practical to add them after the triangulate modifier.

Reproduction :

Using the included simple blend file -

Toggle the triangulate modifier at the end of the stack.
Observe the changes in vertex nomals

TriangulateNormals.blend

Operating system: Windows 10 Pro Graphics card: Nvidia Geforce 1070 blender-2.80.0-893fa598319e-windows64 Adding a triangulate modifer breaks any custom vertex normals, such as those created by the weighted normal modifier. It's typical in game art to add a non-destructive triangulate before exporting to a game engine or texture baking app. Weighted normals behave differently on a triangulated mesh, so it's not practical to add them after the triangulate modifier. Reproduction : Using the included simple blend file - 1. Toggle the triangulate modifier at the end of the stack. 2. Observe the changes in vertex nomals [TriangulateNormals.blend](https://archive.blender.org/developer/F6712649/TriangulateNormals.blend)

Nick Sawyer commented

2019-02-25 18:11:09 +01:00

Added subscriber: @CheeseOnToast

Sebastian Parborg commented

2019-02-26 15:19:20 +01:00

Added subscribers: @ideasman42, @ZedDB

Sebastian Parborg self-assigned this 2019-02-26 15:19:20 +01:00

Sebastian Parborg commented

2019-02-26 15:19:20 +01:00

@ideasman42 would you accept a toggle to keep the vertex-per-face normals?

Campbell Barton commented

2019-02-26 15:39:07 +01:00

Added subscriber: @mont29

Campbell Barton commented

2019-02-26 15:39:07 +01:00

Think this is more for @mont29, would assume if the data is there is should be maintained without having to add an option unless it's especially slow.

Bastien Montagne commented

2019-02-26 15:46:40 +01:00

Generally this is not considered a bug, rather a low priority TODO.

However in that case it looks like a valid need for that pipeline, so think we can add it (should not be that hard) - as an option indeed, as it will likely have a significant impact on speed of this modifier. ;)

@ZedDB feel free to re-assign to me, or add me as reviewer once you have a patch.

Generally this is not considered a bug, rather a low priority TODO. However in that case it looks like a valid need for that pipeline, so think we can add it (should not be that hard) - as an option indeed, as it will likely have a significant impact on speed of this modifier. ;) @ZedDB feel free to re-assign to me, or add me as reviewer once you have a patch.

Sebastian Parborg commented

2019-02-26 15:56:28 +01:00

@mont29 I'll try to have it done until next week. You can snatch it from me if you want to. Just let me know before hand.

Jac Rossiter commented

2019-02-26 16:04:09 +01:00

Added subscriber: @JacRossiter

Jac Rossiter commented

2019-02-26 16:04:09 +01:00

It's fantastic this is being talked about. I believe that triangulation at export also breaks custom normal information. Would this fall into the same scope as the modifier fix? Thanks

Sebastian Parborg commented

2019-02-26 16:10:42 +01:00

I believe that the export, edit mode operator and modifier all share the same base code. I'll get the modifier working first and we can go from there.

michael knubben commented

2019-02-27 14:59:49 +01:00

Added subscriber: @michaelknubben

Bastien Montagne commented

2019-02-28 20:49:07 +01:00

Changed status from 'Open' to: 'Resolved'

Bastien Montagne closed this issue

2019-02-28 20:49:07 +01:00

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

Triangulate modifier breaks custom normals #61942