Skip to content

Hardsuits now block defibulators#2251

Closed
beck-thompson wants to merge 1 commit intoDeltaV-Station:masterfrom
beck-thompson:hardsuits-block-defib
Closed

Hardsuits now block defibulators#2251
beck-thompson wants to merge 1 commit intoDeltaV-Station:masterfrom
beck-thompson:hardsuits-block-defib

Conversation

@beck-thompson
Copy link
Copy Markdown
Contributor

@beck-thompson beck-thompson commented Nov 21, 2024

About the PR

Title! Defibs now are blocked by hardsuits.

Why / Balance

I asked in the discord, majority of people seemed to want it. It makes sense and will make medical have to think about defibbing fully armored security.

Also, requires:
space-wizards/space-station-14#31147
To be merged before it works! (I'll also add comments and localization stuff when that happens as I want it fully merged before doing testing and stuff)

Media

todo

Requirements

  • I have tested all added content and changes.
  • I have added media to this PR or it does not require an ingame showcase.

Changelog

🆑

  • add: Hardsuits now block defibrillators

@beck-thompson beck-thompson requested a review from a team November 21, 2024 05:09
@beck-thompson beck-thompson requested a review from a team as a code owner November 21, 2024 05:09
@beck-thompson beck-thompson marked this pull request as draft November 21, 2024 05:10
@github-actions github-actions Bot added S: Needs Review Awaiting review from a Maintainer Changes: YML Changes any yml files Changes: C# Changes any cs files labels Nov 21, 2024
@BlitzTheSquishy
Copy link
Copy Markdown
Contributor

Gonna need to make sure that softsuits, (Prisoner, Emergancy, Basic, Clown, Mime, Syndicate) dont, and yes clown and mime are softsuits, not hardsuits (despite their name)

@Lyndomen
Copy link
Copy Markdown
Contributor

💯💯

@beck-thompson
Copy link
Copy Markdown
Contributor Author

Gonna need to make sure that softsuits, (Prisoner, Emergancy, Basic, Clown, Mime, Syndicate) dont, and yes clown and mime are softsuits, not hardsuits (despite their name)

Sounds good, when the required PR actually gets merged I'll do more testing to make sure it works on the right things and stuff

@BlitzTheSquishy
Copy link
Copy Markdown
Contributor

BlitzTheSquishy commented Nov 21, 2024

Gonna need to make sure that softsuits, (Prisoner, Emergancy, Basic, Clown, Mime, Syndicate) dont, and yes clown and mime are softsuits, not hardsuits (despite their name)

Sounds good, when the required PR actually gets merged I'll do more testing to make sure it works on the right things and stuff

Also currently where that's placed- it looks like EVERY outer slot item will block defib (would include vox tank harnesses i believe)

also you should be testing it BEFORE it gets merged

@beck-thompson
Copy link
Copy Markdown
Contributor Author

also you should be testing it BEFORE it gets merged

How? I mean I guess I could like chery pick the required PR ahead of time but I personally don't think that's worth the effort. I mostly made this early so people could give their opinions. If people don't want it, then I wont bother putting the work in finding all the weird hardsuit edge cases

@BlitzTheSquishy
Copy link
Copy Markdown
Contributor

also you should be testing it BEFORE it gets merged

How? I mean I guess I could like chery pick the required PR ahead of time but I personally don't think that's worth the effort. I mostly made this early so people could give their opinions. If people don't want it, then I wont bother putting the work in finding all the weird hardsuit edge cases

pull the merge to your dev or make a dev enviro on wizden (which its already merged) and test there, cause we use similar stuff so


namespace Content.Shared.DeltaV.BlockDefibrillator;

[RegisterComponent, NetworkedComponent]
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

access

{

[DataField]
public string OnBlockPopupMessage = "The defibrillator cannot find a pulse through the armor!";
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

locid + localization


public sealed class BlockDefibrillatorSystem : EntitySystem
{
[Dependency] private readonly SharedPopupSystem _popupSystem = default!;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

just _popup

flatReductions:
Heat: 10 # the average lightbulb only does around four damage!
slots: OUTERCLOTHING
- type: BlockDefibrillator // Delta-V
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

pro


namespace Content.Shared.DeltaV.BlockDefibrillator;

[RegisterComponent, NetworkedComponent]
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Add comments

flatReductions:
Heat: 10 # the average lightbulb only does around four damage!
slots: OUTERCLOTHING
- type: BlockDefibrillator // Delta-V
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Remember to double check all the weird edge cases for hardsuits. There are a lot :(

@beck-thompson
Copy link
Copy Markdown
Contributor Author

This isn't too hard to fix but I have more interesting things I'd rather work on! Anyone feel free to copy / re-open this (The code isn't very good so I wouldn't recommend it but whatever 😆)

LaryNevesPR pushed a commit to LaryNevesPR/Estacao-Andromeda that referenced this pull request Apr 17, 2025
# Description

Adds a number of new ways to juggle minds. The pAI adapter, craftable by
hand, allows a pAI to act as a positronic brain. The machine-man
interface allows positronic brains (or adaptered pAIs) to act as
biological brains. It is a tier 2 research and exclusive to the exosuit
fabricator. Additionally finagles some components so that biological
brains in MMIa can be used to control IPCs.

---

<details><summary><h1>Media</h1></summary>
<p>


![image](https://github.com/user-attachments/assets/45895846-8583-4a72-88a4-956afc1938a1)

![image](https://github.com/user-attachments/assets/35db30b1-b819-4bf1-b90b-d5f564b3e490)

![image](https://github.com/user-attachments/assets/72060231-c4d4-4c74-9491-09e86a74415b)
![a pAI is transferred to an adapter, then a
borg](https://github.com/user-attachments/assets/fbad604d-46b9-4710-a008-f8b40ad82c9e)
![a Thaven and an IPC swap brains with the power of
technology](https://github.com/user-attachments/assets/5b82e308-5479-4808-ba9c-f8b7664dbb2c)

</p>
</details>

---

# Changelog

:cl:
- add: Added machine-man interface
- add: Added pAI adapter
- tweak: Allowed MMIs to function as IPC brains
LaryNevesPR pushed a commit to LaryNevesPR/Estacao-Andromeda that referenced this pull request Apr 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changes: C# Changes any cs files Changes: YML Changes any yml files S: Needs Review Awaiting review from a Maintainer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants