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

Engineering/Atmos requires more admin logging #20952

Open
5 tasks
keronshb opened this issue Oct 12, 2023 · 4 comments
Open
5 tasks

Engineering/Atmos requires more admin logging #20952

keronshb opened this issue Oct 12, 2023 · 4 comments
Labels
D3: Low Difficulty: Some codebase knowledge required. P2: Raised Priority: Item has a raised priority, indicating it might get increased maintainer attention. T: New Feature Type: New feature or content, or extending existing content

Comments

@keronshb
Copy link
Contributor

keronshb commented Oct 12, 2023

  • Engine APC should alert admins on power toggle
  • Particle Accelerator should alert admins when power is adjusted (2 and 3. Should also alert if 2 is left on for too long)
  • Singulo should alert admins if it reaches an automatic escape level (5-7)
  • Field Generators should alert admins on power off & barrier off
  • Plasma pump being toggled on
@keronshb keronshb added P2: Raised Priority: Item has a raised priority, indicating it might get increased maintainer attention. T: New Feature Type: New feature or content, or extending existing content D3: Low Difficulty: Some codebase knowledge required. labels Oct 12, 2023
@Chief-Engineer
Copy link
Contributor

PA does trigger an admin alert when set to an unsafe level. afaik the singulo doesn't actually have an automatic escape level and can be contained as long as it fits. field generator and plasma pump alerts might be too spammy and lead to alert fatigue

@keronshb
Copy link
Contributor Author

PA does trigger an admin alert when set to an unsafe level. afaik the singulo doesn't actually have an automatic escape level and can be contained as long as it fits. field generator and plasma pump alerts might be too spammy and lead to alert fatigue

The proposed change is to have it alert on all PA level changes. But that could clog up the admin log so discussion would be needed.

I would argue that singulo level 5-7 are considered the classic auto escape levels.

I adjusted the Field gen and Plasma pump to specify what toggles they should alert on.

@LankLTE
Copy link
Contributor

LankLTE commented Oct 12, 2023

PA does trigger an admin alert when set to an unsafe level. afaik the singulo doesn't actually have an automatic escape level and can be contained as long as it fits.

The singulo will eat containment fields if it grows too large (power level 2 on the PA)

@daerSeebaer
Copy link
Contributor

daerSeebaer commented Oct 12, 2023

afaik the singulo doesn't actually have an automatic escape level and can be contained as long as it fits

It has. It is defined as SingularityBreachThreshold in the SharedSingularitySystem as "5".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
D3: Low Difficulty: Some codebase knowledge required. P2: Raised Priority: Item has a raised priority, indicating it might get increased maintainer attention. T: New Feature Type: New feature or content, or extending existing content
Projects
None yet
Development

No branches or pull requests

4 participants