Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@panxinmiao
Copy link
Contributor

Support KHR_materials_unlit

This extension defines an unlit shading model for use in glTF 2.0 materials, as an alternative to the Physically Based Rendering (PBR) shading models provided by the core specification.
It is very useful in situations where resources are limited for mobile devices or stylized materials that do not require lighting (such as "anime" or "hand drawn" appearances).

For Pygfx, it basically corresponds to our MeshBasicMaterial.

This example demonstrates the differences in rendering the same model using unlit and standard PBR, with unlit material on the left unaffected by lighting and PBR material on the right affected by lighting.

image

@panxinmiao
Copy link
Contributor Author

Currently, in the example, canceling IBL lighting will throw an exception, See #1005.

@panxinmiao panxinmiao marked this pull request as ready for review February 28, 2025 08:17
@panxinmiao panxinmiao requested a review from Korijn as a code owner February 28, 2025 08:17
@almarklein
Copy link
Member

Currently, in the example, canceling IBL lighting will throw an exception

Is this a new error, or simply this use-case exposing it? In other words, do you want to merge this, or get #1005 fixed first?

@panxinmiao
Copy link
Contributor Author

Yes, this is just a use case that exposes the issue in #1005.
This PR is unrelated to #1005. I think it can be merged first.

@panxinmiao
Copy link
Contributor Author

I was planning to fix the bug in #1005 while I was at it, but I found that it seems to be a bit tricky. 😓

@Korijn Korijn enabled auto-merge (squash) February 28, 2025 12:07
@Korijn Korijn merged commit a0dcd0b into pygfx:main Feb 28, 2025
14 checks passed
@panxinmiao panxinmiao mentioned this pull request Mar 2, 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.

3 participants