Lights Respritening - #5502
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughSummaryResprites the Small Light, Light Tube, Emergency Light, and Strobe Light.
User impactPlayers see cleaner wall-mounted light sprites. Powered lights include built-in fake bloom. Light Tube bloom is wider and less linear after review feedback. Risk areas
Guideline gaps
WalkthroughThe change adds RSI metadata for Starlight lighting textures and updates lighting prototypes to use those resources. Entity behavior and component configuration remain unchanged. ChangesStarlight lighting assets
Estimated code review effort: 2 (Simple) | ~10 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
redmushie
left a comment
There was a problem hiding this comment.
I'm sorry but the fake bloom is just way too much. As someone with bad eyesight it unironically makes me think I need to squint to see correctly, but I'm seeing fine already, it's just the sprite.
I don't know if it needs less opacity or harder opacity falloff or smaller size but it needs to be waves hands around less.
|
r4r |
redmushie
left a comment
There was a problem hiding this comment.
I looked it up and what I'm thinking of is astigmatism. It's a condition that makes lights look like this:
It affects 30 to 40% of people, which includes me.
It's apparently caused by having multiple focal points, which is why the light tube bloom specifically is making my brain think I should squint to focus:
I don't have any problems with the light bulb however; I think the main problem with the tube for me is that the bloom is almost in a straight line. Do you think you could make it less straight-line-esque?
I'm speculating here but maybe bleed out a bit wider, as the tube itself illuminates in a circle anyways.
|
github services are down, rerun tests later |
|
I'm gonna be honest, a shader that does this sort of thing would be a more robust method of having light bloom or light rays rather than a sprite. Using a sprite to fake the bloom "works" but isn't really optimal at all. |
maybe, but nobody in all of SS14 has coded one ever even in AGPL forks. Maybe an obscure Russian server has it but if they do their repo is private. |
|
So much better, thank you |
5fb1ade














Short description
Resprites the Small Light, Light Tube, Emergency Light, and Strobe Light. New sprites are a little cleaner, less pre-color tinted so coloring shows up better, and have more built in fake bloom in their on states.
Why we need to add this
Respriting progress. Looks nice.
Media (Video/Screenshots)
Checks
Changelog
🆑 Conflee