-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Rendering artifacts on flat surfaces #12109
Comments
@javagl or @bertt, can you provide a Sandcastle for testing? In the glTF PBR Extensions Sandcastle, if I set "Direct lighting only" and look at the "Water Bottle" model from the side opposite the light direction, I see some artifacts that appear to be reflections from the wrong side of the surface: These go away if I change these lines in float computeDirectSpecularStrength(vec3 normal, vec3 lightDirection, vec3 viewDirection, vec3 halfwayDirection, float alphaRoughness)
{
float NdotL = dot(normal, lightDirection);
float NdotV = abs(dot(normal, viewDirection));
// ... to |
The link to the PBR extensions sandcastle is https://sandcastle.cesium.com/index.html?src=glTF%20PBR%20Extensions.html They can be found in the Sandcastle/Demo that @bertt linked to in the forum thread. But since they depend on the exact view direction and the time of the day (!), here is everything frozen in a standalone sandcastle: For the Water Bottle, there's a bit of noise, but that's probably some roughness from the texture. The glTF in the sandcastle above just has a plain metallic-roughness, so the artifacts look like sharp lines. But from the overall appearance and the "behavior" of the artifacts when zooming/moving, I'd say that the reason is very likely the same in both cases. |
What happened?
There are some strange rendering artifacts that appear on flat surfaces. They look like "screen-space" lines that depend on the view direction.
Reproduction steps
Start the first sandcastle from https://community.cesium.com/t/debug-lines-in-cesium-120/34306 , and rotate the view.
The comment at https://community.cesium.com/t/debug-lines-in-cesium-120/34306/4 summarizes the issue and the commit where it likely has been introduced.
Sandcastle example
See the linked forum thread
Environment
Browser: All
CesiumJS Version: Problem appears in 1.120. It does not appear in 1.119
Operating System: All
The text was updated successfully, but these errors were encountered: