Do not recalculate normals when rendering. #33009

New Issue

Alexander Trum · 2012-10-29T09:47:14+01:00

Alexander Trum commented

2012-10-29 09:47:14 +01:00

%%%Patch introducing a switch which prevents the recalculation of mesh normals when rendering.%%%

Alexander Trum commented

2012-10-29 09:47:14 +01:00

Changed status to: 'Open'

Steffen Dünner commented

2012-11-12 14:16:28 +01:00

%%%Please, please, please include this patch into trunk!%%%

Andy Davies commented

2012-11-12 15:21:34 +01:00

%%%This would be very useful for game devs. who are making foliage. Please merge to trunk :)%%%

Steffen Dünner commented

2012-11-12 15:37:24 +01:00

%%%And even more important for rendering models coming from CAD packages.%%%

Alexander Trum commented

2012-11-12 16:26:26 +01:00

%%%One of our users found a case where the normals are recalculated even with the patch.
So I am working on an enhanced patch now.
I will merge to trunk as soon as I am successful.%%%

%%%One of our users found a case where the normals are recalculated even with the patch. So I am working on an enhanced patch now. I will merge to trunk as soon as I am successful.%%%

Jiri Caslava commented

2012-12-16 13:26:11 +01:00

%%%It is global preference switch?
I think better solution is to make switch within mesh RNA, so both auto recalculated and manual normals mesh objects will be possible in scene.%%%

%%%It is global preference switch? I think better solution is to make switch within mesh RNA, so both auto recalculated and manual normals mesh objects will be possible in scene.%%%

Alexander Trum commented

2013-02-21 15:34:16 +01:00

%%%Now I had the time to enhance the patch. Normals will now be preserved when switching to the edit mode, too. I will attach the new patch file.

I am very new in Blender. So I do not know what Mr. Caslava means with "I think better solution is to make switch within mesh RNA ...". So i still have a global preference switch for preserving the original normals.

Furthermore I have made the changes in Blender 2.63a.
Can anybody help me with merging to trunk?%%%

%%%Now I had the time to enhance the patch. Normals will now be preserved when switching to the edit mode, too. I will attach the new patch file. I am very new in Blender. So I do not know what Mr. Caslava means with "I think better solution is to make switch within mesh RNA ...". So i still have a global preference switch for preserving the original normals. Furthermore I have made the changes in Blender 2.63a. Can anybody help me with merging to trunk?%%%

Alexander Trum commented

2013-02-21 15:36:31 +01:00

%%%Now the new patch file.%%%

Brecht Van Lommel commented

2013-02-21 16:46:12 +01:00

%%%I can see this functionality is useful but a proper implementation is more complicated than this patch.

This shouldn't be a user preference, but rather a setting on a mesh datablock.
This patch only modifies two functions, I don't think this will reliably disable normal map calculation in all places and will still calculate them when needed. Did you go over all mesh related tools and check if they work as expected with this patch?
New vertices or faces should still get their normals calculated?
There shouldn't be printf debug messages left in the patch%%%

%%%I can see this functionality is useful but a proper implementation is more complicated than this patch. * This shouldn't be a user preference, but rather a setting on a mesh datablock. * This patch only modifies two functions, I don't think this will reliably disable normal map calculation in all places and will still calculate them when needed. Did you go over all mesh related tools and check if they work as expected with this patch? * New vertices or faces should still get their normals calculated? * There shouldn't be printf debug messages left in the patch%%%

Alexander Trum commented

2013-02-21 17:09:16 +01:00

%%%1. I'm new to Blender. Is there an example how to add a new setting to the mesh datablock?

I haven't checked all Blender source (i'm a newbie). We have only used the patched Blender intensively.
If this can happen, then I have to check, if normals already exist or not.
Sorry.

What must I provide? How can we proceed?%%%

%%%1. I'm new to Blender. Is there an example how to add a new setting to the mesh datablock? 2. I haven't checked all Blender source (i'm a newbie). We have only used the patched Blender intensively. 3. If this can happen, then I have to check, if normals already exist or not. 4. Sorry. What must I provide? How can we proceed?%%%

$N\A$

$N\A$ N\A commented

2013-06-04 23:38:46 +02:00

%%%I'm also interested in this patch, if any bump is worth the effort.%%%

Bastien Montagne commented

2014-08-15 19:40:14 +02:00

Changed status from 'Open' to: 'Archived'

Bastien Montagne closed this issue

2014-08-15 19:40:14 +02:00

Bastien Montagne self-assigned this 2014-08-15 19:40:14 +02:00

Bastien Montagne commented

2014-08-15 19:40:14 +02:00

Thinks it’s time to archive, we have custom normals support (internal/cycles included) incoming now…

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

Do not recalculate normals when rendering. #33009