Improve our attempt at exporting SpatialMaterials. #341
Conversation
This improves our attempt at converting several material types (Principled, Diffuse, and Emission) to SpatialMaterials. It still leaves a lot to be desired, but implements more than our current approach. What works: - Principled albedo, metallic, roughness, anisotropy, emission, and clearcoat. Principled does not have emission strength, so we just assume 1. - Emission strength and color - Diffuse color and roughness. Fixes godotengine#332.
Everywhere we currently export a color we use mathutils.Color, which doesn't support alpha. This adds an RGBA type we can use to export colors with alpha. Maybe we could use this in other places, but I'm not sure the implications right now.
|
Relates to godotengine/godot#34281. Currently |
If you create a file |
|
Probablyl better for @Jason0214 to review this, but nominally it looks good to me. |
| @@ -57,8 +57,57 @@ def export_material(escn_file, export_settings, bl_object, material): | |||
|
|
|||
| def export_as_spatial_material(material_rsc_name, material): | |||
| """Export a Blender Material as Godot Spatial Material""" | |||
| out = material.node_tree.get_output_node("ALL") | |||
There was a problem hiding this comment.
Maybe add a check for node_tree, it can be None sometimes.
Other changes looks very good. Thank you very much!.
|
Thank you very much! I will fix the regression. |
It still leaves a lot to be desired, but implements more than our current
approach.
What works:
emission, and clearcoat. Principled does not have emission strength, so we
just assume 1.
Fixes #332.
I'm not sure how to set up the test to use the "Export as Spatial" option.
I thought in the past that the export settings were saved in the file,
but it keeps resetting for me now.