Normal map node produces wrong result after reflection fix patch #56209

New Issue

Yusuf Umar · 2018-08-02T09:58:43+02:00

Yusuf Umar commented

2018-08-02 09:58:43 +02:00

System Information
Windows 10 64bit & Radeon RX 480

Blender Version
Broken: After this commit d6e769d32e
Worked: Before that commit

Short description of error
After patch D2574 merged, normal map node produces very inaccurate results. For example in this screenshots below.
Before patch:
After patch:
Normal map used:
I know reflection fix can be useful in certain cases, but I think it should be optional. It can be a toggle or enum on the normal map node itself.

Blend file:
cycles_normal_map_bug.blend

**System Information** Windows 10 64bit & Radeon RX 480 **Blender Version** Broken: After this commit d6e769d32e79 Worked: Before that commit **Short description of error** After patch [D2574](https://archive.blender.org/developer/D2574) merged, normal map node produces very inaccurate results. For example in this screenshots below. Before patch:![normal_map_before_patch.png](https://archive.blender.org/developer/F4106045/normal_map_before_patch.png) After patch:![normal_map_after_patch.png](https://archive.blender.org/developer/F4106037/normal_map_after_patch.png) Normal map used:![normal_map_.png](https://archive.blender.org/developer/F4106049/normal_map_.png) I know reflection fix can be useful in certain cases, but I think it should be optional. It can be a toggle or enum on the normal map node itself. Blend file: [cycles_normal_map_bug.blend](https://archive.blender.org/developer/F4106067/cycles_normal_map_bug.blend)

Yusuf Umar commented

2018-08-02 09:58:43 +02:00

Added subscriber: @ucupumar

blender-admin commented

2018-08-02 09:58:43 +02:00

#100420 was marked as duplicate of this issue

blender-admin commented

2018-08-02 09:58:43 +02:00

#67891 was marked as duplicate of this issue

Yusuf Umar changed title from ~~Normal map produces wrong result after reflection fix patch~~ to Normal map node produces wrong result after reflection fix patch

2018-08-02 10:03:00 +02:00

Lukas Stockner was assigned by Bastien Montagne

2018-08-02 17:29:30 +02:00

Yusuf Umar commented

2018-11-02 07:49:05 +01:00

After D3816 patch, it still produce wrong result compared to Blender 2.79. I still think this reflection fix should be optional.

After [D3816](https://archive.blender.org/developer/D3816) patch, it still produce wrong result compared to Blender 2.79. I still think this reflection fix should be optional. ![normal_map_after_newest_patch.jpg](https://archive.blender.org/developer/F5333856/normal_map_after_newest_patch.jpg)

Brecht Van Lommel commented

2019-08-01 18:59:29 +02:00

Added subscribers: @EugenioPignataro, @dr.sybren, @NahuelBelich

Lukas Stockner was unassigned by Dalai Felinto

2019-12-23 16:36:03 +01:00

Dalai Felinto commented

2019-12-23 16:36:03 +01:00

Added subscriber: @LukasStockner

Sybren A. Stüvel commented

2020-02-11 11:58:28 +01:00

Changed status from 'Confirmed' to: 'Needs User Info'

Sybren A. Stüvel commented

2020-02-11 11:58:28 +01:00

I can't reproduce this issue with the latest Blender (master @ be8879718e); I get the "before patch" result.

@ucupumar Please re-test this with the latest Blender build from https://builder.blender.org/ and let us know if this is still an issue. If it is, please provide clear steps to reproduce.

I can't reproduce this issue with the latest Blender (master @ be8879718e24e417d299eb298b8a9d4d2ca324ee); I get the "before patch" result. @ucupumar Please re-test this with the latest Blender build from https://builder.blender.org/ and let us know if this is still an issue. If it is, please provide clear steps to reproduce.

Yusuf Umar commented

2020-02-11 12:26:57 +01:00

I've already tested with the latest build and the bug is still there, you should render the scene using cycles.

Philipp Oeser commented

2020-02-11 13:20:15 +01:00

Added subscriber: @lichtwerk

Philipp Oeser commented

2020-02-11 13:20:15 +01:00

Changed status from 'Needs User Info' to: 'Confirmed'

Philipp Oeser commented

2020-02-11 13:20:15 +01:00

In #56209#549017, @ucupumar wrote:
After D3816 patch, it still produce wrong result compared to Blender 2.79. I still think this reflection fix should be optional.

@dr.sybren: I am also seeing the above ^
So will confirm, not sure though if this should be classified a bug?

note: since this is about ensure_valid_reflection, this is also an issue with it #57109 (Flickering artifacts in (animated) DOF areas)

> In #56209#549017, @ucupumar wrote: > After [D3816](https://archive.blender.org/developer/D3816) patch, it still produce wrong result compared to Blender 2.79. I still think this reflection fix should be optional. > ![normal_map_after_newest_patch.jpg](https://archive.blender.org/developer/F5333856/normal_map_after_newest_patch.jpg) @dr.sybren: I am also seeing the above ^ So will confirm, not sure though if this should be classified a bug? note: since this is about `ensure_valid_reflection`, this is also an issue with it #57109 (Flickering artifacts in (animated) DOF areas)

Yusuf Umar commented

2020-02-11 13:45:21 +01:00

@lichtwerk I think it still classified as bug since it's a wrong normal result for a standard 3d application. Currently there's no way to get correct normal map value using cycles. My suggestion is to give checkbox on normal map node to use correct normal map or normal map with reflection fix.

Sybren A. Stüvel commented

2020-02-11 15:41:06 +01:00

Removed subscriber: @dr.sybren

Sybren A. Stüvel commented

2020-02-11 15:41:06 +01:00

I'll defer to the Cycles team for further handling of this.

Roman commented

2020-05-05 13:31:59 +02:00

Added subscriber: @rowvn

Michael Möller commented

2020-10-09 11:04:41 +02:00

Added subscriber: @mmoeller

Vladislav Macíček commented

2021-05-23 02:45:17 +02:00

Added subscriber: @Aeraglyx

Vladislav Macíček commented

2021-05-23 04:12:49 +02:00

Hi, just wanted to say it's still an issue in some cases. For example when you use Bump node to get a derivative of something and use that value for a math operation. In that case you need the actual normal.
Or to give a concrete example, there's a method for even line thickness using Bump node - which only works in Eevee (as Luca Rood and Kolupsy noticed) because of this.

So I agree with Yusuf - would it be possible to add something like a Limit Angle checkbox that's perhaps checked on by default?

Hi, just wanted to say it's still an issue in some cases. For example when you use *Bump* node to get a derivative of something and use that value for a math operation. In that case you need the actual normal. Or to give a concrete example, there's a method for even line thickness using *Bump* node - which only works in Eevee (as Luca Rood and Kolupsy noticed) because of this. ![Untitled Project_Untitled Project_blender_sInYFt_2021-05-23_03.09.35.png](https://archive.blender.org/developer/F10136500/Untitled_Project_Untitled_Project_blender_sInYFt_2021-05-23_03.09.35.png) So I agree with Yusuf - would it be possible to add something like a *Limit Angle* checkbox that's perhaps checked on by default?

Peter Sandeld commented

2021-06-27 16:02:55 +02:00

Added subscriber: @MaxIT

Peter Sandeld commented

2021-06-27 16:02:55 +02:00

This comment was removed by @MaxIT

*This comment was removed by @MaxIT*

Peter Sandeld commented

2021-06-27 16:12:43 +02:00

Hi,

This has become a frustrating problem for me personally as a 3D creator.

Normal input reflections are not showing correctly in both EEVEE and Cycles.

It appears as though the normal reflections are not showing correctly for normal angles steeper than 45 degrees.

An example of this is to use a spherical normal map connected to the Principled BSDF normal input, and having only the metallic slider at full strength (fully reflective).

You will then see the reflections are not showing the outer 3rd of the spheres reflections correctly, or the normal reflections steeper than 45 degrees; much the same as the incorrect reflections seen in Yusuf Umar's images posted here showing incorrect reflections, only it becomes clearer when fully reflective with an HDRI environment.

The same results can be seen by procedurally generating spherical normal maps, or other shapes, such as a cylinder or sloped edge with some normal angles steeper than 45 degrees.

From testing, normal reflections are not showing correctly for versions 2.91.2 / 2.93.0 / 3.0 Alpha.

Would be great if this can be fixed.

PeteS

Hi, This has become a frustrating problem for me personally as a 3D creator. Normal input reflections are not showing correctly in both EEVEE and Cycles. It appears as though the normal reflections are not showing correctly for normal angles steeper than 45 degrees. An example of this is to use a spherical normal map connected to the Principled BSDF normal input, and having only the metallic slider at full strength (fully reflective). You will then see the reflections are not showing the outer 3rd of the spheres reflections correctly, or the normal reflections steeper than 45 degrees; much the same as the incorrect reflections seen in Yusuf Umar's images posted here showing incorrect reflections, only it becomes clearer when fully reflective with an HDRI environment. The same results can be seen by procedurally generating spherical normal maps, or other shapes, such as a cylinder or sloped edge with some normal angles steeper than 45 degrees. From testing, normal reflections are not showing correctly for versions 2.91.2 / 2.93.0 / 3.0 Alpha. Would be great if this can be fixed. PeteS

Eugenio Pignataro commented

2021-07-23 17:08:16 +02:00

Yes, we've the same problem.
I made a water drops seamless,

2.79 cycles

2.79 blender engine

THANKS!!!!!

Yes, we've the same problem. I made a water drops seamless, 2.79 cycles ![Screenshot from 2021-07-23 12-05-56.png](https://archive.blender.org/developer/F10235035/Screenshot_from_2021-07-23_12-05-56.png) 2.79 blender engine ![Screenshot from 2021-07-23 12-05-18.png](https://archive.blender.org/developer/F10235034/Screenshot_from_2021-07-23_12-05-18.png) THANKS!!!!!

Sam Morse-Brown commented

2021-09-08 21:04:09 +02:00

Added subscriber: @ParallelMayhem