Skip to content

[MIRROR] Ambience Buzz Handling Changes + Ambience buzz requires enviorment power #4183

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

Merged
merged 2 commits into from
Jul 3, 2024

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#3440
Original PR: tgstation/tgstation#84479

About The Pull Request

Partial port of DaedalusDock/daedalusdock#996

  • Fixes ambience prefrence inconsistancy issues with Observers unable to turn off the ambience buzz.
  • Ambience buzz requires a working, existing, charged APC with power to the enviorment.
  • Moves update_ambience_area() and refresh_looping_ambience() into code\controllers\subsystem\ambience.dm for better organization.

Why It's Good For The Game

Mostly to tackle a prefrence bug where ghosts are unable to turn off ambience buzz. But this also includes ambience buzz requiring a powered area. Makes the station feel more dead when there's no enviorment power. There should also be no buzzing inside space.

Changelog

🆑 Kapu (ported by StrangeWeirdKitten)
fix: Ambience buzz will now respect ship ambience prefrences for observers.
sound: Ambience buzz requires APC enviorment power to function
/:cl:

…orment power [MDB IGNORE] (#3440)

* Ambience Buzz Handling Changes + Ambience buzz requires enviorment power

* conflict

---------

Co-authored-by: The Sharkening <[email protected]>
Co-authored-by: Fluffles <[email protected]>
@Steals-The-PRs Steals-The-PRs added Mirroring conflict git cherry-pick во что-то уткнулся. Не ставить вручную, только для бота TG Mirror labels Jul 1, 2024
@Iajret Iajret enabled auto-merge July 2, 2024 22:34
@Iajret Iajret merged commit a6a0fd7 into master Jul 3, 2024
26 checks passed
AnywayFarus added a commit that referenced this pull request Jul 3, 2024
@Iajret Iajret deleted the upstream-mirror-3440 branch July 3, 2024 17:20
Iajret pushed a commit that referenced this pull request Aug 8, 2024
* 'Cybersun Training Sim' DM arena

* deco update

* lol oops

* make it prettier

* One Lattice

* floor prettier

* last thing probably

* embarrassing dont open

* medkits actually are full

* why did this have so many issues
Iajret pushed a commit that referenced this pull request Aug 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Mirroring conflict git cherry-pick во что-то уткнулся. Не ставить вручную, только для бота TG Mirror
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants