First draft of KHR_materials_diffuse_transmission. #179
Conversation
|
Looks like a model with this extension got added to glTF-Sample-Assets although I'm not sure what this means for the status of the extension (I'm used to the sample repositories only getting extensions after they get merged to glTF repository but the extension PR is still open): https://github.com/KhronosGroup/glTF-Sample-Assets/blob/main/Models/MandarinOrange/glTF/MandarinOrange.gltf |
|
|
||
| // Defaults | ||
| cgltf_fill_float_array(out_diff_transmission->diffuse_transmission_color_factor, 3, 1.0f); | ||
| out_diff_transmission->diffuse_transmission_factor = 0.f; |
There was a problem hiding this comment.
nit, 0.f will be assigned automatically as we zero out all fields by default
| @@ -1911,6 +1921,11 @@ void cgltf_free(cgltf_data* data) | |||
| cgltf_free_extensions(data, data->materials[i].iridescence.iridescence_texture.extensions, data->materials[i].iridescence.iridescence_texture.extensions_count); | |||
| cgltf_free_extensions(data, data->materials[i].iridescence.iridescence_thickness_texture.extensions, data->materials[i].iridescence.iridescence_thickness_texture.extensions_count); | |||
| } | |||
| if (data->materials[i].has_diffuse_transmission) | |||
| { | |||
| cgltf_free_extensions(data, data->materials[i].diffuse_transmission.diffuse_transmission_texture.extensions, data->materials[i].diffuse_transmission.diffuse_transmission_texture.extensions_count); | |||
There was a problem hiding this comment.
These two calls should use cgltf_free_texture_view for now (and the PR needs to be rebased to latest master) to take care of extras. This will keep the code correct for now, although once I get to #118 this will become unnecessary as the code will just get deleted.
Adds support for KHR_materials_diffuse_transmission.
Not ready for merge just yet
The specification for this extension is currently marked as "Draft" and is not yet ratified, though it appears to be getting close. Staging this change in case anyone else is looking to take a peak and plug into their own renderers. I've gone so far as to just make sure this compiles and verified a basic model's properties are read from this extension. More testing is needed.