Skip to content

Conversation

@MiiBond
Copy link
Contributor

@MiiBond MiiBond commented Oct 28, 2025

Adding support to import and export KHR_materials_coat, the new coating extension for glTF that supports additional features like coat color, coat darkening and coat anisotropy.

I also removed the previous sub-extensions that I had added recently (e.g. KHR_materials_clearcoat_darkening, etc.) as they are no longer needed.

I also used the texture merger to combine the coat weight and coat roughness textures, if needed.

@bjsplat
Copy link
Collaborator

bjsplat commented Oct 28, 2025

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

@bjsplat
Copy link
Collaborator

bjsplat commented Oct 28, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Oct 28, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Oct 28, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Oct 28, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Oct 29, 2025

@deltakosh deltakosh requested a review from bghgary October 29, 2025 20:59
@bjsplat
Copy link
Collaborator

bjsplat commented Oct 30, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Oct 30, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Oct 30, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Oct 30, 2025

@deltakosh deltakosh enabled auto-merge (squash) October 30, 2025 22:38
Copy link
Contributor

@bghgary bghgary left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks so much simpler now that it's one extension.

* @returns A new, internal texture with the coat weight in the red channel and coat roughness in the green channel
* @internal
*/
async function CreateMergedCoatInternalTexture(babylonMaterial: OpenPBRMaterial): Promise<Nullable<InternalTexture>> {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: we usually put Async postfix for async functions as a convention

Suggested change
async function CreateMergedCoatInternalTexture(babylonMaterial: OpenPBRMaterial): Promise<Nullable<InternalTexture>> {
async function CreateMergedCoatInternalTextureAsync(babylonMaterial: OpenPBRMaterial): Promise<Nullable<InternalTexture>> {

@bjsplat
Copy link
Collaborator

bjsplat commented Oct 30, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Oct 30, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Oct 30, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Oct 30, 2025

auto-merge was automatically disabled October 31, 2025 05:46

Head branch was pushed to by a user without write access

@bjsplat
Copy link
Collaborator

bjsplat commented Oct 31, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Oct 31, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Oct 31, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Oct 31, 2025

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants