Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WORKING Directional Lighting in 2024 #10474

Merged
merged 30 commits into from
Aug 7, 2024

Conversation

Tsar-Salat
Copy link
Contributor

@Tsar-Salat Tsar-Salat commented Jan 14, 2024

Ports:

About The Pull Request

This ports the directional lighting system from Teegee. Its a recoup of #7418, and #8093

Essentially, this adds a cone in front of the lightsource, rather than a radius of light. This is meant to emulate lights like headlights or flashlights.

Radius-based light sources & directional light sources are defined by its MOVABLE_LIGHT define

TODO

  • fix ethereals mutcolors getting reset to white
  • fix weird light toggling bug

Why It's Good For The Game

This is a direct upgrade of our lighting system.

As we tend towards projects like #10473, I think this is a good complement to what it is trying to achieve, which is why I am reviving this PR

Testing Photographs and Procedure

Screenshots&Videos
dreamseeker_ZkNp5kL7RA.mp4

Working Gunlights

dreamseeker_f6TSk2wbMx.mp4

Working Ethereals

image

Changelog

🆑 rkz, azarak, TemporalOroboros, Rohesie, Couls, ShizCalev, TastyFish
add: Adds directional lighting system in 2024!
add: adds directional lighting to cyborg lights, flashlights, and gunlights
refactor: refactors glow effects to be less terrible. Includes ethereal glow, and also the glowy/antiglowy mutation.
/:cl:

@MarkusLarsson421
Copy link
Contributor

Correct me if I am wrong, but it looks like it is just off-setting the light source a few meters in front of where you are facing?

@Crossedfall Crossedfall added the Frozen Frozen until a larger project is finished label Jan 31, 2024
@Tsar-Salat
Copy link
Contributor Author

oops lol

Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot added Mapping DMM Change GitHub Config Update Sound Administration Tools TGUI-Changes Contains changes to TGUI. Make sure its up to date with TGUI 4.0 labels Feb 29, 2024
@Tsar-Salat Tsar-Salat force-pushed the directional_lighting_rkz branch from 4eb2329 to 6b6cf00 Compare February 29, 2024 03:03
@github-actions github-actions bot removed Merge Conflict GitHub Config Update Administration TGUI-Changes Contains changes to TGUI. Make sure its up to date with TGUI 4.0 labels Feb 29, 2024
@PowerfulBacon PowerfulBacon added the Test Merged This PR is currently in rotation label Jul 30, 2024
Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@Rukofamicom
Copy link
Contributor

What's up with mineral scan pulse?

@Rukofamicom Rukofamicom added Test Merged This PR is currently in rotation and removed Test Merged This PR is currently in rotation labels Aug 4, 2024
Copy link

github-actions bot commented Aug 4, 2024

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@Rukofamicom Rukofamicom added Test Merged This PR is currently in rotation and removed Test Merged This PR is currently in rotation labels Aug 4, 2024
@Rukofamicom
Copy link
Contributor

damn conflicts can't make up their mind and neither can I because of them

Copy link

github-actions bot commented Aug 6, 2024

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@Rukofamicom Rukofamicom added this pull request to the merge queue Aug 7, 2024
@Rukofamicom
Copy link
Contributor

In the event this doesn't merge cleanly, ping for re-attempt please.
This is very prone to conflicts so we should get it in as soon as we can.

I may be busy at the moment and unable to respond, but I can push the merge button even remotely if it's not currently conflicted

@Rukofamicom Rukofamicom added Testmerge Passed Seemed fine in testmerge and removed Test Merged This PR is currently in rotation labels Aug 7, 2024
Merged via the queue into BeeStation:master with commit 6cdb7ea Aug 7, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants