Skip to content

Disable rim lighting (r_rimLighting) by default#1896

Merged
slipher merged 1 commit intoDaemonEngine:masterfrom
slipher:norim
Feb 23, 2026
Merged

Disable rim lighting (r_rimLighting) by default#1896
slipher merged 1 commit intoDaemonEngine:masterfrom
slipher:norim

Conversation

@slipher
Copy link
Member

@slipher slipher commented Feb 13, 2026

"Rim lighting" purports to make the edges of models stand out. However in my tests, I don't see a very strong correlation between the parts which are lit up and the edges of models. For more geometrically complex models such as the Barricade, it sometimes randomly lightens the model all over the place, which seems bad.

No rim lighting:

unvanquished-atcshd-under-fog-barricade
unvanquished-plat23-dlight-psaw1

With rim lighting:

unvanquished-atcshd-under-fog-barricade

unvanquished-plat23-dlight-psaw1

If the idea is accepted, I will also make a PR for the graphics presets.

"Rim lighting" purports to make the edges of models stand out. However
in my tests, I don't see a very strong correlation between the parts
which are lit up and the edges of models. For more geometrically complex
models such as the Barricade, it sometimes randomly lightens the model
all over the place, which looks bad.
@slipher
Copy link
Member Author

slipher commented Feb 20, 2026

I will try an experiment to make it easier to evaluate renderer PRs like this: I will upload the screenshot comparison tool and the screenshot data set I'm using to unvanquished.net for reviewers to browse. Of course the downside is that the thread is not self-contained; I will probably delete the screenshots later.

Navigate using the arrow keys: https://users.unvanquished.net/~slipher/screenshot-compare/1896/viewer.html. Note that fog is disabled since I had several otherwise useful test scenes where the model was obscured by fog.

@illwieckz
Copy link
Member

Quote from Chat:

<illwieckz> can bad rim lighting be related to some other bad normal stuff?
<perturbed-mob> Nah it’s just a shitty algorithm
<perturbed-mob> Only works if the model is real smooth and convex
<perturbed-mob> For example it works on granger’s butt but not barricade

Copy link
Member

@illwieckz illwieckz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@illwieckz
Copy link
Member

If the feature is not suitable for Unvanquished, we may also remove the option from the GUI.

@slipher slipher merged commit 4c90dfa into DaemonEngine:master Feb 23, 2026
9 checks passed
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