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

[Port-Ish] Adds new station alert levels to the game + code fix about alerts #12254

Open
wants to merge 20 commits into
base: master
Choose a base branch
from

Conversation

XeonMations
Copy link
Contributor

@XeonMations XeonMations commented Feb 10, 2025

About The Pull Request

Adds multiple station alert levels to the game, such as:

Yellow Alert: Used for engineering emergencies.
Cyan Alert: Used for medical emergencies.
Black Alert: Used for when the station is under active siege. Such as nuke ops. Activates automatically when warops are declared.
Epsilon Alert: Used for when a deathsquad is incoming.
Lambda Alert: Used for when heretics ascend, blood or clock cults win.
Gamma Alert: Centcom's "We're taking over control" alert. Used for when command is being bad actors (mostly)

Audio sources/PRs:

And makes it so you cant lower the alert from anything higher than red. Stops shenanigens such as delta alert > green on nukie rounds.

Ports the following PR:

Why It's Good For The Game

Station alert levels should be more than just security issues.
The RBMK exploded? CBRN threat, call in code cyan.
The station's cut in half? Code yellow.
Nukies? Oh shit, that's a code black.

Testing Photographs and Procedure

Screenshots&Videos
2025-02-11.04-09-02.mp4

Changelog

🆑 XeonMations
fix: You can no longer change the alert when the alert is higher than red alert.
add: Added Yellow alert, used for engineering emergencies.
add: Added Cyan alert, used for medical emergencies.
add: And a couple more station alerts, that are not able to be activated manually as a player, to the game.
/:cl:

@XeonMations XeonMations changed the title [Port-Ish] Adds new station alert levels to the game [Port-Ish] Adds new station alert levels to the game + code fix about alerts Feb 10, 2025
@Marshmellow105
Copy link
Contributor

Marshmellow105 commented Feb 10, 2025

Gamma seems pointless, if an ERT is sent, its sent based on current alert (amber ERT, blue ERT, red ERT)

at that, what is a medical emergency aside from a disease outbreak?

@XeonMations
Copy link
Contributor Author

XeonMations commented Feb 10, 2025

at that, what is a medical emergency aside from a disease outbreak?

Zombies, blob, a virus, maybe xenos, so on.

@XeonMations
Copy link
Contributor Author

XeonMations commented Feb 10, 2025

Gamma seems pointless, if an ERT is sent, its sent based on current alert (amber ERT, blue ERT, red ERT)

Mostly to serve as more of a reminder to the station that "HELP IS ON THE WAY", if the station's requested help, it's more than likely they are on red alert or blue anyhow.

@Marshmellow105
Copy link
Contributor

Marshmellow105 commented Feb 10, 2025

reminder to the station that "HELP IS ON THE WAY"

Wouldn't an announcement serve the same purpose without treading the alert level SOP line?

If we're on blue, something is crazy so we call an ERT, it goes to 'gamma' and now im not sure what powers I hold to search as a security officer bc it isnt blue anymore.

Being that alerts are tied with SOP, it might be wise to ask the headmins what they think aswell.

@TsunamiAnt
Copy link
Contributor

we will 100% need new SOP for all of these, but that is fine, i love the flexibility

@TsunamiAnt
Copy link
Contributor

oh, as a suggestion for epsilon, how about having the alert be more of a "station is now under central command" like the announcement says and less of a strict dsquad thing?

Cuz like Deathsquads are so fucking rare, but the announcement itself would still be great for say, a CC authorized leadership purge, or something like that. "all contracts terminated" is very restrictive.

@github-actions github-actions bot added Config Update TGUI-Changes Contains changes to TGUI. Make sure its up to date with TGUI 4.0 labels Feb 10, 2025
@SunShine214
Copy link
Contributor

Have you tested if lowering delta alert can still happen if the threat is gone? (AI died before doomsday, nuke deactivated)

@XeonMations XeonMations marked this pull request as ready for review February 10, 2025 19:57
Copy link
Member

@PowerfulBacon PowerfulBacon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd like for yellow and cyan to be merged so that to the player they both display as yellow alert, and this alert is for when a department needs to take command of all crew.

They can stay different alert levels in code but should both show as yellow with something like yellow - engineering or yellow - medical emergency.

The sub message for the yellow alert should specifically say that the head of that department gains authority over all crew and ignoring their orders makes you subject to arrest.

Other than that, I don't mind admin only alerts.

Copy link
Member

@PowerfulBacon PowerfulBacon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yellow medical should still show in a yellow alert box in my opinion, since in SOP it will be classified under yellow alert.

Comment on lines 239 to 244
/datum/config_entry/string/alert_yellow
config_entry_value = "The security alert level has been changed to yellow (Engineering). There is currently a critical engineering issue on the station. Crewmembers are advised to avoid and report damaged areas. All crew are instructed to obey all instructions given by the Chief Engineer for the duration of this alert."

/datum/config_entry/string/alert_cyan
config_entry_value = "The security alert level has been changed to yellow (Medical). There is an ongoing C-B-R-N threat to the station. Crewmembers are advised to don protective gear, and personal oxygen systems until the alert is lifted. Report any symptoms to medical personnel immediately. All crew are instructed to obey all instructions given by the Chief Medical Officer for the duration of this alert."

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can the path of these be something like alert_engineering and alert_medical? There may come a point where we want new ones for other departments and it makes the path more clear

@XeonMations
Copy link
Contributor Author

Yellow medical should still show in a yellow alert box in my opinion, since in SOP it will be classified under yellow alert.

It's cyan because light blue colors are associated with medical.

@PowerfulBacon
Copy link
Member

PowerfulBacon commented Feb 14, 2025

Also not a big fan of the yellow alert sound shown in testing evidence, its very high pitch and annoying

@PowerfulBacon
Copy link
Member

It's cyan because light blue colors are associated with medical.

The actual alert has become yellow though and the current text in that alert is illegible due to the colour scheme

@XeonMations
Copy link
Contributor Author

It's cyan because light blue colors are associated with medical.

The actual alert has become yellow though and the current text in that alert is illegible due to the colour scheme

image

@TsunamiAnt
Copy link
Contributor

https://wiki.beestation13.com/view/User:PowerfulBacon/Standard_Operating_Procedure#Code_Gamma_-_Command_Restructuring

Code gamma should not take command from the captain, since it's more of a "ERT is present, assist them if so ordered" alert.

"command restructure" is just basically what epsilon is after all.

@TsunamiAnt
Copy link
Contributor

TsunamiAnt commented Feb 15, 2025

wait no it was changed? why is gamma, the sound explicitly not mentioning central command takeover, now the CC takeover?

image

okay you might want to take a second look at all of this lmao. As long as the "Attention, station now under central command" sound is actually doing what it says, i am happy. What CC then does with the control should be vague, whether command purge or full d-squad.

@XeonMations
Copy link
Contributor Author

XeonMations commented Feb 15, 2025

As long as the "Attention, station now under central command" sound is actually doing what it says, i am happy.

Bacon asked me to make gamma station "The station is now under direct central command control"

Deathsquad alert is now Epsilon. As requested.

All of this will probably change even more in the future, due to the headmins having to confirm the SOP changes that bacon has written in here: https://wiki.beestation13.com/view/User:PowerfulBacon/Standard_Operating_Procedure

@TsunamiAnt
Copy link
Contributor

yeah literally disregard all i said, just naming confusions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Fix Sound TGUI-Changes Contains changes to TGUI. Make sure its up to date with TGUI 4.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants