Maniphest T93798

Eevee: Group node shaders do not get compiled when using Mix Shader node
Closed, Resolved
Actions

Assigned To
Jacques Lucke (JacquesLucke)
Authored By
Andy Goralczyk (eyecandy)
Dec 7 2021, 10:27 AM
Tags
  • BF Blender
  • EEVEE & Viewport
Subscribers
Aaron Carlisle (Blendify)
Andy Goralczyk (eyecandy)
Jacques Lucke (JacquesLucke)
Philipp Oeser (lichtwerk)
Vitor Boschi (vitorboschi)

Description

System Information
Operating system: Linux-5.13.0-7620-generic-x86_64-with-glibc2.33 64 Bits
Graphics card: NVIDIA GeForce RTX 3080/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 470.82.00

Blender Version
Broken: version: 3.1.0 Alpha, branch: master, commit date: 2021-12-07 07:55, hash: rB1de363662427
Worked: hash: 7c4fc5b58d2e

Caused by rBd5efda72f501: Cleanup: Migrate all shader nodes to c++

Short description of error
I have a file with a group node that outputs a Shader. In it, there are two shaders which get combined with a Mix Shader node. The resulting shader is completely black in Rendered and Material view. Last thursday (see hash mentioned above) this was still working.

Exact steps for others to reproduce the error
See .blend file:

eevee_mix_group.blend1 MBDownload

The monkey is completely black, previously the shader compiled fine. If you ungroup the node it works again.

Related Objects

Mentioned In
rB0f48b37aae02: Revert moving all shader nodes to c++
D13498: Fix: Revert moving shader nodes to C++.
T93797: Crash [or faulty eevee render] when opening UDIM_monster
Mentioned Here
rB0f48b37aae02: Revert moving all shader nodes to c++
T93797: Crash [or faulty eevee render] when opening UDIM_monster
rBd5efda72f501: Cleanup: Migrate all shader nodes to c++

Event Timeline

Andy Goralczyk (eyecandy) created this task.Dec 7 2021, 10:27 AM
Philipp Oeser (lichtwerk) changed the task status from Needs Triage to Confirmed.Dec 7 2021, 11:11 AM
Philipp Oeser (lichtwerk) triaged this task as High priority.
Philipp Oeser (lichtwerk) added projects: EEVEE & Viewport, Nodes.
Philipp Oeser (lichtwerk) added subscribers: Aaron Carlisle (Blendify), Philipp Oeser (lichtwerk).

Might have the same roots as T93797: Crash [or faulty eevee render] when opening UDIM_monster.
It is for sure caused by the same commit rBd5efda72f501: Cleanup: Migrate all shader nodes to c++.

Will keep reports separate for now though.

CC @Aaron Carlisle (Blendify)

Philipp Oeser (lichtwerk) updated the task description.Dec 7 2021, 11:11 AM
Vitor Boschi (vitorboschi) added a subscriber: Vitor Boschi (vitorboschi).Dec 7 2021, 11:29 AM

I have seen this too. You don't even need the mix shader to trigger the bug, just having a single shader inside a node group is enough

Jesse Yurkovich (deadpin) mentioned this in T93797: Crash [or faulty eevee render] when opening UDIM_monster.Dec 7 2021, 12:23 PM
Julian Eisel (Severin) mentioned this in D13498: Fix: Revert moving shader nodes to C++..Dec 7 2021, 12:54 PM
Jacques Lucke (JacquesLucke) mentioned this in rB0f48b37aae02: Revert moving all shader nodes to c++.Dec 7 2021, 1:28 PM
Jacques Lucke (JacquesLucke) closed this task as Resolved.Dec 7 2021, 2:33 PM
Jacques Lucke (JacquesLucke) claimed this task.
Jacques Lucke (JacquesLucke) added a subscriber: Jacques Lucke (JacquesLucke).

Fixed by rB0f48b37aae0230119523a9718b331961bd989a78.

Hans Goudey (HooglyBoogly) removed a project: Nodes.May 16 2022, 11:01 AM