Cycles material sometimes overrides GLSL material functionality #38633

New Issue

Jonatan Bijl · 2014-02-14T09:57:55+01:00

Jonatan Bijl commented

2014-02-14 09:57:55 +01:00

System Information
WinXP 32 bits (but not really important I think)

Blender Version
Broken: 2.69

Short description of error
Some GLSL materials have shading and shadows, others dont. My guess is that for some materials it uses the BI material as a basis for the GLSL, and for others it uses the Cycles material as basis for the GLSL. I use materials that have both a Cycles and a BI node tree, so that very late in the process I can still choose which renderer to use.

Exact steps for others to reproduce the error
Open this file: GLSLmaterial.blend
The two tiles have a very similar material, but one has a shadow the other does not.
In http://blenderartists.org/forum/showthread.php?327207-GLSL-materials-with-or-without-shadows someone suggests removing and re-adding the cycles output node as a workaround.

**System Information** WinXP 32 bits (but not really important I think) **Blender Version** Broken: 2.69 **Short description of error** Some GLSL materials have shading and shadows, others dont. My guess is that for some materials it uses the BI material as a basis for the GLSL, and for others it uses the Cycles material as basis for the GLSL. I use materials that have both a Cycles and a BI node tree, so that very late in the process I can still choose which renderer to use. ![glslmaterial.jpg](https://archive.blender.org/developer/F77233/glslmaterial.jpg) **Exact steps for others to reproduce the error** Open this file: [GLSLmaterial.blend](https://archive.blender.org/developer/F77235/GLSLmaterial.blend) The two tiles have a very similar material, but one has a shadow the other does not. In http://blenderartists.org/forum/showthread.php?327207-GLSL-materials-with-or-without-shadows someone suggests removing and re-adding the cycles output node as a workaround.

Jonatan Bijl commented

2014-02-14 09:57:55 +01:00

Changed status to: 'Open'

Jonatan Bijl commented

2014-02-14 09:57:55 +01:00

Added subscriber: @jonim8or

Brecht Van Lommel was assigned by Sergey Sharybin

2014-02-14 09:59:31 +01:00

Sergey Sharybin commented

2014-02-14 09:59:31 +01:00

Added subscriber: @Sergey

Jonatan Bijl commented

2014-02-14 10:16:22 +01:00

Hypothesis: It looks like it is using the first output it finds in m.nodes, and generates a shader for that.

In the example file Maps/Basalt is the material that works OK and Maps/Concrete is the one that has no shadows.

  >>> dict((m.name, list(m.node_tree.nodes)) for m in D.materials)
  {
  'Maps/Basalt': [bpy.data...nodes["Output"], bpy.data...nodes["Diffuse BSDF"], bpy.data...nodes["Material Output"], bpy.data...nodes["Material"], bpy.data...nodes["Image Texture"]],
  'Maps/Concrete': [bpy.data...nodes["Material Output"], bpy.data...nodes["Diffuse BSDF"], bpy.data...nodes["Material"], bpy.data...nodes["Output"], bpy.data...nodes["Image Texture"]]}

and after the workaround of deleting and re-adding the cycles output, in both node lists the Output is first, and therefore the BI material is used as a basis.:

  >>> dict((m.name, list(m.node_tree.nodes)) for m in D.materials)
  {
  'Maps/Basalt': [bpy.data...nodes["Output"], bpy.data...nodes["Diffuse BSDF"], bpy.data...nodes["Material Output"], bpy.data...nodes["Material"], bpy.data...nodes["Image Texture"]],
  'Maps/Concrete': [bpy.data...nodes["Diffuse BSDF"], bpy.data...nodes["Material"], bpy.data...nodes["Output"], bpy.data...nodes["Image Texture"], bpy.data...nodes["Material Output"]]}

proposed/expected way for it to work:
When in BI mode, use BI output node, when in Cycles mode use Cycles output node

Hypothesis: It looks like it is using the first output it finds in m.nodes, and generates a shader for that. In the example file Maps/Basalt is the material that works OK and Maps/Concrete is the one that has no shadows. ``` >>> dict((m.name, list(m.node_tree.nodes)) for m in D.materials) { 'Maps/Basalt': [bpy.data...nodes["Output"], bpy.data...nodes["Diffuse BSDF"], bpy.data...nodes["Material Output"], bpy.data...nodes["Material"], bpy.data...nodes["Image Texture"]], 'Maps/Concrete': [bpy.data...nodes["Material Output"], bpy.data...nodes["Diffuse BSDF"], bpy.data...nodes["Material"], bpy.data...nodes["Output"], bpy.data...nodes["Image Texture"]]} ``` and after the workaround of deleting and re-adding the cycles output, in both node lists the Output is first, and therefore the BI material is used as a basis.: ``` >>> dict((m.name, list(m.node_tree.nodes)) for m in D.materials) { 'Maps/Basalt': [bpy.data...nodes["Output"], bpy.data...nodes["Diffuse BSDF"], bpy.data...nodes["Material Output"], bpy.data...nodes["Material"], bpy.data...nodes["Image Texture"]], 'Maps/Concrete': [bpy.data...nodes["Diffuse BSDF"], bpy.data...nodes["Material"], bpy.data...nodes["Output"], bpy.data...nodes["Image Texture"], bpy.data...nodes["Material Output"]]} ``` proposed/expected way for it to work: When in BI mode, use BI output node, when in Cycles mode use Cycles output node

blender-admin commented

2014-02-14 15:11:58 +01:00

This issue was referenced by blender/blender-addons-contrib@a252e27fec

This issue was referenced by blender/blender-addons-contrib@a252e27fec16e09efe873a6bbbd29b3ad5d4c67f

blender-admin commented

2014-02-14 15:11:58 +01:00

This issue was referenced by a252e27fec

This issue was referenced by a252e27fec16e09efe873a6bbbd29b3ad5d4c67f

Brecht Van Lommel commented

2014-02-14 15:12:15 +01:00

Fixed now, thanks for the report.