Skip to content

Conversation

@yaRnMcDonuts
Copy link
Member

@yaRnMcDonuts yaRnMcDonuts commented Mar 10, 2025

This PR attempts to fix the issue that my previous PR unfortunately failed to fix: #2365

I have made some more minor changes to the shader code that might fix it.

And shortly I'm also going to update this PR so that the PBRTerrain tests will allow users to easily toggle the DebugValuesMode debug view of each individual pbr layer so that issues like this are easier to troubleshoot in the future.

…rendering issue

This PR attempts to fix the issue that my previous PR unfortunately failed to fix: #2365

I have made some more minor changes to the shader code that might fix it.

But I'm also going to update the PBRTerrain tests so users can easily toggle the DebugValuesMode debug view of each individual pbr layer so that issues like this are easier to troubleshoot in the future.
@yaRnMcDonuts yaRnMcDonuts changed the title Add debugging to PBRTerrainTests to to help troubleshoot an unsolved … Add debugging to PBRTerrainTests to troubleshoot rendering issue Mar 10, 2025
@yaRnMcDonuts yaRnMcDonuts added this to the v3.8.0 milestone Mar 19, 2025
@yaRnMcDonuts
Copy link
Member Author

This PR will be merged in about 24 hours.

This did not fix the bug unfortunately. but the debug updates to the pbr terrain examples will still be useful in the future.

@yaRnMcDonuts yaRnMcDonuts merged commit 4f055d1 into master Mar 20, 2025
15 checks passed
@yaRnMcDonuts yaRnMcDonuts deleted the yaRnMcDonuts-patch-2 branch March 20, 2025 08:01
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.

2 participants