Skip to content
This repository was archived by the owner on May 22, 2025. It is now read-only.

Conversation

@JohnFulpWillard
Copy link
Contributor

@JohnFulpWillard JohnFulpWillard commented Oct 8, 2023

Document the changes in your pull request

Ports tgstation/tgstation#54520
Ports tgstation/tgstation#52881
Ports tgstation/tgstation#60239

Ports tgstation/tgstation#53239
Ports tgstation/tgstation#53816
Ports tgstation/tgstation#52393

This is refactoring many things about the lighting system (and opacity), which I did while trying to fix Ethereal lights (which I failed at doing- for some reason it seems to work until power goes to 2< like it's got some hardcap on and I can't figure out why that is the case). I thought I should throw this stuff up anyways because why not.

It removes an undocumented change in #17271 about some lights not contributing to lumcount, idk why that was added but if it's a problem I can re-add. I mostly just tried to get it to be as close to TG's as possible so future work on it wouldn't be as bad.

This also adds directional lights, I'm not sure if it's something that is wanted so for now I only added it to seclites. The support is there, so you can add it to wherever you want after idk I don't want to feature code.

Why is this good for the game?

Hopefully bringing lighting closer to TG's system will make future portings easier and allow Ethereals to finally work.

Testing

image

Spriting

Wiki Documentation

Changelog

🆑
rscadd: Seclites now have directional lights- point them in the direction you want to see more of.
experimental: Ported over many refactors to lighting and opacity. Please report if you can walk or see through walls unintentionally.
/:cl:

@JohnFulpWillard JohnFulpWillard requested a review from a team as a code owner October 8, 2023 13:45
@Yogbot-13 Yogbot-13 added the Feature This adds new content to the game label Oct 8, 2023
@GDRdev
Copy link

GDRdev commented Oct 8, 2023

pain

@ThatLing
Copy link
Contributor

ThatLing commented Oct 8, 2023

BeeStation tried porting this multiple times and had to revert each time because of various issues, did you look into these?

It removes an undocumented change in #17271 about some lights not contributing to lumcount, idk why that was added but if it's a problem I can re-add

BeeStation/BeeStation-Hornet#6092

@iloveloopers
Copy link
Contributor

Preternis buff

@JohnFulpWillard
Copy link
Contributor Author

It removes an undocumented change in #17271 about some lights not contributing to lumcount, idk why that was added but if it's a problem I can re-add

BeeStation/BeeStation-Hornet#6092

The PR doesn't mention adding this so I think it was completely undocumented, I dont know if it was discussed at all at the time of adding but it seemed like stealth balancing stuff in a refactor pr. The PR linked has 3 TG PRs as "being ported", you're linking Bee.

@JohnFulpWillard
Copy link
Contributor Author

For the issues, I have no way to check if flashlights break late into the round without seeing this as at least a testmerge. For the light in the corner, I remember this being an issue for some time on TG as well, but it's been fixed now.
For the issue on ethereal lights breaking, our ethereal lights are already broken and has a bandaid on top to make them work. This doesn't fix it or break it any further.

Yogbot-13 added a commit that referenced this pull request Oct 12, 2023
Yogbot-13 added a commit that referenced this pull request Oct 12, 2023
Yogbot-13 added a commit that referenced this pull request Oct 12, 2023
@Moltijoe Moltijoe added Awaiting - Action - HD Awaiting action from Jamie Awaiting - Action - Maintainer This PR is awaiting an action from a maintainer and removed Awaiting - Action - HD Awaiting action from Jamie labels Oct 16, 2023
@MajManatee MajManatee added Awaiting - Action - Sprite Review Awaiting sprite review from sprite maintainers and removed Awaiting - Action - Maintainer This PR is awaiting an action from a maintainer labels Oct 17, 2023
@Moltijoe Moltijoe added the Merge conflict Files are conflicting with current live changes. label Oct 19, 2023
@JohnFulpWillard JohnFulpWillard marked this pull request as draft October 21, 2023 05:24
@JohnFulpWillard
Copy link
Contributor Author

I gotta figure out why replays won't respect it's alpha

@JohnFulpWillard JohnFulpWillard marked this pull request as ready for review November 9, 2023 22:41
@Moltijoe Moltijoe added Merge conflict Files are conflicting with current live changes. and removed Merge conflict Files are conflicting with current live changes. labels Nov 20, 2023
@Moltijoe Moltijoe removed Awaiting - Action - Sprite Review Awaiting sprite review from sprite maintainers Merge conflict Files are conflicting with current live changes. labels Nov 25, 2023
@Moltijoe Moltijoe merged commit 5be3c1d into yogstation13:master Nov 25, 2023
Yogbot-13 added a commit that referenced this pull request Nov 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Feature This adds new content to the game

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants