Skip to content

Fix vertex-lit surfaces with material system#1398

Merged
illwieckz merged 1 commit intoDaemonEngine:masterfrom
VReaperV:fix-material-vertex-light
Oct 28, 2024
Merged

Fix vertex-lit surfaces with material system#1398
illwieckz merged 1 commit intoDaemonEngine:masterfrom
VReaperV:fix-material-vertex-light

Conversation

@VReaperV
Copy link
Copy Markdown
Contributor

Don't overwrite tess.svars.color by Tess_ComputeColor() after setting it with SetVertexLightingSettings(). Fixes some surfaces being whitened out, like crates, bricks and boxes on terminus.

@slipher
Copy link
Copy Markdown
Member

slipher commented Oct 27, 2024

LGTM

@illwieckz
Copy link
Copy Markdown
Member

Does it also fix r_lightMode 1 with material system?

@VReaperV
Copy link
Copy Markdown
Contributor Author

...was it broken? I would need to check because I wasn't aware of that.

@VReaperV
Copy link
Copy Markdown
Contributor Author

Does it also fix r_lightMode 1 with material system?

Just tried and it does, indeed, fix that.

@illwieckz
Copy link
Copy Markdown
Member

illwieckz commented Oct 28, 2024

Yes, I noticed it was broken, but I didn't find time to report it before with the release rush, and since r_lightMode 1 is meant to be used in presets for hardware not supporting the material system, and because material system is experimental and not enabled by default, it was not blocking the release. Thanks for fixing that! 😀️

@VReaperV VReaperV force-pushed the fix-material-vertex-light branch from 73440bc to 61365d6 Compare October 28, 2024 17:24
@VReaperV
Copy link
Copy Markdown
Contributor Author

Rebasing first because CI failed to start for some reason.

@VReaperV VReaperV force-pushed the fix-material-vertex-light branch from 9a8a1fa to 10d9f42 Compare October 28, 2024 17:33
@illwieckz illwieckz merged commit e0c03c1 into DaemonEngine:master Oct 28, 2024
@VReaperV VReaperV deleted the fix-material-vertex-light branch October 28, 2024 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants