Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mention per-vertex shading is currently unimplemented in Standard Material 3D #9096

Conversation

Calinou
Copy link
Member

@Calinou Calinou commented Mar 13, 2024

This also removes an outdated mention of Oren-Nayar diffuse mode, which no longer exists since 4.0.

For reference, see godotengine/godot#43093 and godotengine/godot#83360.

@Calinou Calinou added bug area:manual Issues and PRs related to the Manual/Tutorials section of the documentation cherrypick:4.2 labels Mar 13, 2024
Copy link
Contributor

@skyace65 skyace65 left a comment

Choose a reason for hiding this comment

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

Thew new image needs to be a webp. Everything else looks good.

Copy link
Member

@mhilbrunner mhilbrunner left a comment

Choose a reason for hiding this comment

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

Besides the mentioned WebP conversion, this looks good.

…erial 3D

This also removes an outdated mention of Oren-Nayar diffuse mode,
which no longer exists since 4.0.
@Calinou Calinou force-pushed the standardmaterial3d-no-per-vertex-shading branch from 5f52795 to 7a7f815 Compare March 23, 2024 17:16
@mhilbrunner
Copy link
Member

We had first approval, how about second approval. Thanks, merging! And thanks for taking care of the conversion.

@mhilbrunner mhilbrunner merged commit b6c5dd8 into godotengine:master Mar 24, 2024
1 check passed
mhilbrunner added a commit that referenced this pull request May 7, 2024
…x-shading

Mention per-vertex shading is currently unimplemented in Standard Material 3D

(cherry picked from commit b6c5dd8)
@mhilbrunner
Copy link
Member

Cherrypicked to 4.2.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:manual Issues and PRs related to the Manual/Tutorials section of the documentation bug topic:rendering
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Standard Material 3D page has outdated lighting information
3 participants