Paradise icon indicating copy to clipboard operation
Paradise copied to clipboard

Lamp Bloom, Exposure and Glare

Open MrRomainzZ opened this issue 1 year ago • 29 comments

What Does This PR Do

Adapts https://github.com/TauCetiStation/TauCetiClassic/pull/11943

Requires DB update (at least, if no one tells me how to avoid it)

Adds new effects for lamps by adding bloom, exposure and glare. Everything may be toggled on or off. Admins even have Bloom Edit button in debug section to test different parameters (or to play with players)

Why It's Good For The Game

Beauty

Images of changes

https://youtu.be/KuCSgDfvQ3M

v2 https://youtu.be/1cOyj2OkLEA

v3 Comment below

Testing

A lot on my localhost

Changelog

:cl: add: Added togglable through preferences bloom, exposure and glare effects on lamps /:cl:

MrRomainzZ avatar Mar 10 '24 13:03 MrRomainzZ

That looks amazing! Does it have any performance impact?

meow20 avatar Mar 10 '24 13:03 meow20

@meow20 I guess it does, but I haven't noticed any on my GTX 650 Ti :) And you can turn it off whenever you'd like

MrRomainzZ avatar Mar 10 '24 13:03 MrRomainzZ

I believe that the glare could be better adjusted to the light's brightness. There's a few awkward scenarios such as the Paramedic's room where the light is somewhat dim but the glare cuts through it with a large brightness

image

Arthri avatar Mar 10 '24 13:03 Arthri

vM0aVX

m-dzianishchyts avatar Mar 10 '24 13:03 m-dzianishchyts

This PR is an absolute beauty, I agree with Athri's concerns however, looks particularly awkward in darker rooms.

SynthTwo avatar Mar 10 '24 13:03 SynthTwo

@Arthri I'll look into that after fixed tests

MrRomainzZ avatar Mar 10 '24 13:03 MrRomainzZ

The "circadian rhythms" bull might be active and screwing up things a bit?

meow20 avatar Mar 10 '24 13:03 meow20

Now it's time to fix that brightness...

MrRomainzZ avatar Mar 10 '24 15:03 MrRomainzZ

@Arthri, @SynthTwo, what will you say about this one? Changed glare logic a bit and tweaked lights https://youtu.be/1cOyj2OkLEA

MrRomainzZ avatar Mar 10 '24 19:03 MrRomainzZ

Awesome work!

Arthri avatar Mar 11 '24 09:03 Arthri

Looks better to me, also glad to hear that the lighting is also toggleable.

SynthTwo avatar Mar 11 '24 18:03 SynthTwo

Glare now doesn't affect color of lamps and very subtle. Bloom-sprite of lamp is now semi-transparent.

Here are screenshots: Day Shift / Night Shift / Fire Alarm / Emergency Lights Bar: изображение изображение изображение изображение

Medbay Reception: изображение изображение изображение изображение

Morgue: изображение изображение изображение изображение

MrRomainzZ avatar Mar 14 '24 21:03 MrRomainzZ

Just to note, we still not resolved problems with lighting effects above objects. It can be noticeable if, for example, you have a tree or something else blocking the lamp. TG solved similar problems with emissive lighting/planes. Before we adapted some similar solution, our compromise was that we can live with it and just do some map fixes.

And we also dimmed our base light globally for better effect and replaced floors with more clean ones

If you interested

изображение

volas avatar Apr 28 '24 15:04 volas

This pull request seems to be stale as there have been no changes in 14 days, please make changes within 7 days or the PR will be closed. If you believe this is a mistake, please inform a development team member on Discord.

github-actions[bot] avatar May 15 '24 01:05 github-actions[bot]

2+ month... Sad

AyIong avatar May 16 '24 16:05 AyIong

This pull request seems to be stale as there have been no changes in 14 days, please make changes within 7 days or the PR will be closed. If you believe this is a mistake, please inform a development team member on Discord.

github-actions[bot] avatar May 31 '24 01:05 github-actions[bot]

I think I might rip the SQL changes from this PR into their own PR so this can be TM'd easier.

AffectedArc07 avatar May 31 '24 18:05 AffectedArc07

I'm done with current conversations (except that layers issue). Need reviews

MrRomainzZ avatar Jun 02 '24 13:06 MrRomainzZ

image image sometring really works weird

UPD: turning on off Toggle New Lighting fixes that

HMBGERDO avatar Jun 18 '24 12:06 HMBGERDO

image image sometring really works weird

UPD: turning on off Toggle New Lighting fixes that

I've encountered this once and cleared byond cache after that issue. Since then haven't seen it again. Don't know what's causing this, but I'll look into the code for clues

MrRomainzZ avatar Jun 18 '24 13:06 MrRomainzZ

Hm. Found out that this is how exposure renders when looked from the camera as well. Will fix

MrRomainzZ avatar Jun 18 '24 17:06 MrRomainzZ

I've seen one runtime so far:

image

SteelSlayer avatar Jun 18 '24 20:06 SteelSlayer

the wall lights don't update in all shuttles that rotate them (noticeable in the metastation emergency shuttle)

Cokecolagua avatar Jun 18 '24 22:06 Cokecolagua

I've seen one runtime so far:

image

expanding on this. The runtime is due to people spray painting lights, and it runtimes ALOT. pqMk7TsO8m

1080pCat avatar Jun 21 '24 19:06 1080pCat

I've seen one runtime so far: image

expanding on this. The runtime is due to people spray painting lights, and it runtimes ALOT. pqMk7TsO8m

Can you assume another reason of runtime? I'm spray painting lights but don't get those...

MrRomainzZ avatar Jun 22 '24 02:06 MrRomainzZ

But I think I have some sort of a solution to this...

MrRomainzZ avatar Jun 22 '24 02:06 MrRomainzZ

!merge_upstream

DGamerL avatar Jun 29 '24 01:06 DGamerL

Merging upstream failed, see the action run log for details: https://github.com/ParadiseSS13/Paradise/actions/runs/9720633441

paradisess13[bot] avatar Jun 29 '24 01:06 paradisess13[bot]

Some lights permanantly lose glare after being broken, and replaced by hand with a new light. image

ATP-Engineer avatar Jul 03 '24 20:07 ATP-Engineer