search

KhronosGroup / glTF-Blender-IO

Blender glTF 2.0 importer and exporter
https://docs.blender.org/manual/en/latest/addons/import_export/scene_gltf2.html
Apache License 2.0
1.48k stars 318 forks source link

Export: emission materials export too bright #1301

Closed scurest closed 2 years ago

scurest commented 3 years ago

A basic emission material like this

emis

exports as

  "materials": [
    {
      "emissiveFactor": [1, 0, 0],
      "pbrMetallicRoughness": {},
      ...
    }

Since pbrMetallicRoughness is omitted, the defaults are baseColor=[1,1,1,1], metallic=1, roughness=1. This is much too bright.

out

Messing around in Blender to find an equivalent-looking Principled node, baseColor=[0,0,0,1], metallic=(anything), roughness=1 is much better.

Note: This material setup was discussed at https://github.com/KhronosGroup/glTF-Blender-IO/pull/1209#issuecomment-693649108.

emackey commented 3 years ago

Good catch, the exporter should produce baseColor: [0, 0, 0, 1] for materials like this.

julienduroure commented 2 years ago

@scurest @emackey Can you please have a look on #1569 ?