Skip to content

Conversation

abaire
Copy link
Contributor

@abaire abaire commented Apr 1, 2025

Fixes lighting in cases where ambient and/or emissive colors are taken from vertex colors rather than the material.

Tests

PR diff results

@abaire
Copy link
Contributor Author

abaire commented Apr 1, 2025

Emissive color is still slightly off, but it's much closer to correct than before. Will keep in draft for a bit to see if I can figure out why, but it'd be interesting to test this against games that have issues with emissive textures.

@Triticum0
Copy link
Collaborator

Triticum0 commented Apr 2, 2025

Might fix Constantine lighting issues #1201 Tested, no change in issue

@abaire abaire force-pushed the fix_material_color_source_ambient_and_emissive branch 2 times, most recently from d70d2a0 to 4ec7cce Compare April 2, 2025 15:41
@abaire abaire marked this pull request as ready for review April 2, 2025 16:59
@abaire
Copy link
Contributor Author

abaire commented Apr 2, 2025

Fixed the delta in emissive handling, should be ready for review now.

@abaire abaire force-pushed the fix_material_color_source_ambient_and_emissive branch from 4ec7cce to 6829a30 Compare April 17, 2025 02:46
@abaire abaire force-pushed the fix_material_color_source_ambient_and_emissive branch 2 times, most recently from cfb1962 to 90fd122 Compare June 3, 2025 19:34
@abaire abaire force-pushed the fix_material_color_source_ambient_and_emissive branch 2 times, most recently from 8385aa3 to bf68b27 Compare June 28, 2025 14:07
@abaire abaire force-pushed the fix_material_color_source_ambient_and_emissive branch from bf68b27 to a8cb91c Compare July 2, 2025 17:08
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