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

Airlock Seals 2: I'm dumb and a fool edition #53114

Merged
merged 35 commits into from Sep 13, 2020

Conversation

Jimmius
Copy link
Contributor

@Jimmius Jimmius commented Aug 21, 2020

About The Pull Request

This is basically just an exact remake of #52595 except without all the bumbling fuckups I made due to being dumb. Mega thanks to Rohesie for helping me, considering literally all of the problems were ones I made for myself (the merge conflict was literally due to a missing cricket bat icon). In conclusion, I'm dumb.

Adds door seals to the game, inspired by the ones used in Alien Isolation and SOMA. Trivially easy for a human to remove, but makes it a nightmare to try and smash through if you're an alien, an evil robot, or don't have hands. I also slightly changed how airlock reinforcement works, as now there are two ways to increase the integrity, the previous way of just resetting the max_integrity after the removal of the plating wouldn't work.

Why It's Good For The Game

Makes it harder for aliens, cyborgs, and simplemobs to get into places the crew don't want them, while being easy to remove by humans.

Changelog

🆑
add: Adds door seals to the game, printable at engineering and security lathes after they've been researched. Makes airlocks much harder to smash down if you can't remove them.
tweak: Slightly changed how bonus health to an airlock works
/:cl:

Jimmius added 21 commits July 26, 2020 15:06
I'll be amazed if this actually works, but lets give it a punt
You can no longer infinitely remove seals to weaken airlocks
Also changes the return from a failed attempt to TRUE, so a failed attempt returns correctly and we don't continue attempting to open the door (previously this would trigger the safety lock on an unpowered arrivals door despite it being sealed)
basically just changes "this airlock" to "[src]"
… maybe nulls the variables, if I understood what that means correctly
…appveyor and travis to shit themselves

also adds a thing to stop message spam if you unseal constantly
if you think it shouldn't then yell at me and I'll undo it
@tgstation-server tgstation-server added Feature Exposes new bugs in interesting ways Sprites A bikeshed full of soulless bikes. Tweak labels Aug 21, 2020
@actioninja
Copy link
Member

The main issue I take with this from a design standpoint is that there's zero trade-off to them. There's no real reason to not just put these on literally every door and provide pretty objective advantages.

@ATH1909
Copy link
Contributor

ATH1909 commented Aug 21, 2020

The main issue I take with this from a design standpoint is that there's zero trade-off to them. There's no real reason to not just put these on literally every door and provide pretty objective advantages.

Do you have to take off the seal every time you need to go through the door, then put it back on the door behind you?

@Jimmius
Copy link
Contributor Author

Jimmius commented Aug 21, 2020

Do you have to take off the seal every time you need to go through the door, then put it back on the door behind you?

Yes, there's a 3 second seal time and a 2 second unseal time (I made the unseal shorter so you're always losing ground if you're trying to use these in a chase against a human), so you'd have to wait 5 seconds after doing this to each door. I could up the times to make it more arduous to both apply and remove, if you think it's too quick. Something like 7 seconds seal, 2 seconds unseal (I don't want a long unseal time because that sort of gives them a use for delaying human enemies, which is not the goal) is probably long enough to put people off sealing literally every door on the station.

@tralezab
Copy link
Contributor

Yeah, I would only put these on low traffic, high security or very high danger areas. maybe nuke ops? I dunno

@actioninja
Copy link
Member

I forgot to ever comment again but yeh I actually just completely misunderstood the feature, it actually seems fine.

@cacogen
Copy link
Contributor

cacogen commented Aug 31, 2020

This will make me play Alien: Isolation finally. Fuck aliens and fuck simple mobs.

@anconfuzedrock
Copy link
Contributor

anconfuzedrock commented Sep 7, 2020

wait a second aren't airlock seals basically reinforced tables/ welding doors
gotta be human to climb a table

@Jimmius
Copy link
Contributor Author

Jimmius commented Sep 7, 2020

sort of, a lot of the things this is supposed to counter can just walk over tables anyway, like spiders and xenos and what have you. It also doubles the integrity of the airlock, which welding doesn't do.

code/game/machinery/doors/airlock.dm Outdated Show resolved Hide resolved
code/game/machinery/doors/airlock.dm Outdated Show resolved Hide resolved
code/game/machinery/doors/door.dm Outdated Show resolved Hide resolved
@Timberpoes
Copy link
Member

Timberpoes commented Sep 13, 2020

Geez laweez. Sorry this has been hanging so long. I've now assigned myself to it.

I'll see about getting this merged as soon as. It's definitely overdue and I'm not sure how you slipped under our radars.

Current triage is as follows:

  • Code review -> Minor notes requesting improved documentation.
  • Icon review -> Just noticed the Twaticus seal of approval.
  • Have you tested this? -> While the above is happening, you can help expedite the process by explaining any testing you've done to be sure this feature works as intended.
  • Mergies -> When the above 3 steps are completed, we'll get this in.

@Timberpoes Timberpoes self-assigned this Sep 13, 2020
so it matches the overlay on the door better (i made it  less rectangular by a single pixel)
@Jimmius
Copy link
Contributor Author

Jimmius commented Sep 13, 2020

Things wot I have tested so far:

1: I have used the seal on every single airlock currently in the game (even the double airlock used in ruins). It works and displays correctly, and I can remove it from all of them.
2: Non-Cultists can safely remove this from Cult airlocks without getting bounced, so culties don't get a free lunch (bumping will still bounce).
3: If an airlock with safety unlocks is unpowered and sealed, clicking will remove the seal (you can unlock the safety system once it's removed).
4: The seal keeps the door closed during emags and staff of doors, and blocks attempts to open from the AI and Cyborgs, as well as pulsing the open wire.
5: Everything works fine if you seal AND reinforce and airlock, in either order.

If there's any other edge case interactions that I can't think of, just say and I'll test them.

* Called when attempting to remove the seal from an airlock
*
* Here because we need to call it and return if there was a seal so we don't try to open the door
* or try it's safety lock while it's sealed
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* or try it's safety lock while it's sealed
* or try its safety lock while it's sealed

Copy link
Member

Choose a reason for hiding this comment

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

Yes, I am being that pedantic! Muahahaha.

He, his. She, hers. It, its.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fuck, how did I not notice this

its its its its its
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Exposes new bugs in interesting ways Sprites A bikeshed full of soulless bikes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants