Maniphest T70666

Exporting to FBX ignores the emission socket on the Principled BSDF
Closed, ResolvedTO DO
Actions

Assigned To
Bastien Montagne (mont29)
Authored By
Lewis Toohey (Lewis2e)
Oct 9 2019, 9:27 AM
Tags
  • Add-ons (Community)
  • Import/Export
Subscribers
Bastien Montagne (mont29)
Christopher Ross (mrchrissross)
Germano Cavalcante (mano-wii)
Julien DUROURE (julien)
Lewis Toohey (Lewis2e)

Description

System Information
Operating system: Linux-5.3.5-arch1-1-ARCH-x86_64-with-arch 64 Bits
Graphics card: Mesa DRI Intel(R) HD Graphics 620 (Kaby Lake GT2) Intel Open Source Technology Center 4.5 (Core Profile) Mesa 19.2.0

Blender Version
Broken: version: 2.80 (sub 75), branch: master, commit date: 2019-07-29 14:47, hash: rBf6cb5f54494e
Worked: (optional)

Short description of error
When exporting an object and material to an FBX file, the Emission socket of the Principled BSDF seems to be ignored, when the other sockets are respected.

Exact steps for others to reproduce the error

Open a new blend file. Create a new material for the default cube.

Plug Image Textures into the Base Color, Roughness and Emission sockets of the Principled BSDF node.

Export the object to an fbx file.

(Optionally) delete the object.

Re-import the fbx file.

Examine the material of the imported object. The Image Textures for Base Color and Roughness are preserved. The Image Texture for Emission has been discarded.

Revisions and Commits

rBA Blender Add-ons
rBAdb2c65d9e788 Fix T70666: OBJ IO: Add support for new Emission option of Principled BSDF.
rBAc4f78f14931b Fix T70666: FBX IO: Add support for new emission option of Principled BSDF.

Related Objects

Mentioned In
rB826db891abe8: Node Shader wrapper: add access to the Emission color of Principled BSDF node.

Event Timeline

Lewis Toohey (Lewis2e) created this task.Oct 9 2019, 9:27 AM
Bastien Montagne (mont29) claimed this task.Oct 9 2019, 8:10 PM
Bastien Montagne (mont29) lowered the priority of this task from 90 to Normal.
Bastien Montagne (mont29) edited projects, added Add-ons (Community), Import/Export; removed BF Blender.
Bastien Montagne (mont29) added a subscriber: Bastien Montagne (mont29).

That’s more of a TODO than a bug really, that property did not exist I think when the node wrapper was written… Will add it to the tool and the formats that support it (iirc OBJ/MTL also has some emissive info?).

Bastien Montagne (mont29) edited a custom field.Oct 9 2019, 8:10 PM
Julien DUROURE (julien) added a subscriber: Julien DUROURE (julien).Oct 10 2019, 9:19 AM

Note that glTF needs to be updated too
https://github.com/KhronosGroup/glTF-Blender-IO/issues/462

Bastien Montagne (mont29) added a comment.Oct 10 2019, 5:01 PM

@Julien DUROURE (julien) aye, but iirc glTF uses its own 'translation' system, not the node_shader_utils.py, to get shading values out of a node shader?

Julien DUROURE (julien) added a comment.Oct 10 2019, 5:12 PM

Right, it's not using it for now.
I need to have a look on node_shader_utils.py because I was not really aware it exists.

Bastien Montagne (mont29) changed the task status from Unknown Status to Resolved by committing rBAc4f78f14931b: Fix T70666: FBX IO: Add support for new emission option of Principled BSDF..Oct 10 2019, 5:21 PM
Bastien Montagne (mont29) mentioned this in rB826db891abe8: Node Shader wrapper: add access to the Emission color of Principled BSDF node..
Bastien Montagne (mont29) added a commit: rBAc4f78f14931b: Fix T70666: FBX IO: Add support for new emission option of Principled BSDF..
Bastien Montagne (mont29) added a commit: rBAdb2c65d9e788: Fix T70666: OBJ IO: Add support for new Emission option of Principled BSDF..
Bastien Montagne (mont29) added a comment.Oct 10 2019, 5:25 PM

@Julien DUROURE (julien) yeah, if you could switch to it (and extend it maybe with extra features you may already have in glTF wrapper?), that would be ideal, since that would leave a single place to handle that conversion between nodal shader and 'old', 'fixed pipeline' shading type…

Brecht Van Lommel (brecht) merged a task: T71085: No emission texture export when exported to obj & mtl.Nov 6 2019, 5:04 PM
Brecht Van Lommel (brecht) added subscribers: Christopher Ross (mrchrissross), Germano Cavalcante (mano-wii).