Skip to content

Materials can now specify a shadow attenuation factor#8540

Merged
pixelflinger merged 2 commits intomainfrom
ma/shadow-attenuation
Mar 20, 2025
Merged

Materials can now specify a shadow attenuation factor#8540
pixelflinger merged 2 commits intomainfrom
ma/shadow-attenuation

Conversation

@pixelflinger
Copy link
Collaborator

@pixelflinger pixelflinger commented Mar 18, 2025

Materials have a new property: shadowStrength that can be used to attenuate all shadows received by this material. e.g.:

void material(inout MaterialInputs material) {
  prepareMaterial(material);
  material.shadowStrength = 0.1;
}

FIXES=[391663042]

@pixelflinger pixelflinger added the internal Issue/PR does not affect clients label Mar 18, 2025
@pixelflinger pixelflinger marked this pull request as draft March 18, 2025 23:36
@pixelflinger
Copy link
Collaborator Author

draft because: we will change "shadowAttenuation" to "shadowStrength" and invert its effect

@pixelflinger pixelflinger force-pushed the ma/shadow-attenuation branch from d5d7ec2 to ec66ad9 Compare March 19, 2025 18:44
@pixelflinger pixelflinger marked this pull request as ready for review March 19, 2025 18:50
@pixelflinger pixelflinger force-pushed the ma/shadow-attenuation branch from ec66ad9 to ad41d57 Compare March 19, 2025 18:50
Materials have a new property: shadowStrength that can be used to
attenuate all shadows received by this material. e.g.:

```
void material(inout MaterialInputs material) {
  prepareMaterial(material);
  material. shadowStrength = 0.1;
}
```

FIXES=[391663042]
@pixelflinger pixelflinger force-pushed the ma/shadow-attenuation branch from ad41d57 to f9e7806 Compare March 20, 2025 18:30
Co-authored-by: Powei Feng <powei@google.com>
@pixelflinger pixelflinger merged commit 6b91f30 into main Mar 20, 2025
13 checks passed
@pixelflinger pixelflinger deleted the ma/shadow-attenuation branch March 20, 2025 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Issue/PR does not affect clients

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments