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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removes most tiny fans from Deltastation, replaced with airlock cycle system #28890

Merged
merged 1 commit into from Jun 29, 2017

Conversation

AKlimentyev
Copy link

Tiny fans were always considered bad map design, and only used when absolutely needed.
Mso's lazy airlock cycling is a much better system.

A few were left in in places like the telecomms server room, the toxins launcher, and the old containment area since it seemed okayish to leave them. Those airlocks still got cycling though.

馃啈
del: The majority of the tiny fans on Deltastation have been removed.
tweak: Airlocks going to space, and secure areas like the bridge and sec on Delta have been given the cycling system.
/:cl:

Adds the airlock cycle system to a lot of airlocks
@tgstation-server tgstation-server added Map Edit Removal This was too fun, too fun! I'm turning this feature around labels Jun 27, 2017
@lzimann
Copy link
Contributor

lzimann commented Jun 27, 2017

These tiny fans shouldn't be in the station.

@imtakingabreakdontatme
Copy link
Contributor

Tiny fans were always considered bad map design

Citation needed

@AKlimentyev
Copy link
Author

Couldn't find the pr where they were added to box's engineering to deal with the air being lost.
It's mainly been discussion on coderbus and stuff. Basically, fans are a handwavy cheat item to deal with atmos escaping through doors. They were added to replace the snowflake of the shelter pod doors blocking air. They make no actual sense, as a gust of air shouldn't be able to keep an atmosphere in on one side and hard vacuum on the other side.
For ruins, and such tinyfans are perfectly fine, as they don't have a way to replenish their air supply usually. On a fully functioning station, it's just silly.

@optimumtact
Copy link
Member

They should be on ruins only, not actual station maps

@RandomMarine
Copy link
Contributor

Re-fluff tiny fans to something that makes goofy sci-fi sense.

@Bawhoppen
Copy link
Contributor

I just wanted to make them craftable instead, our atmos is fucked completely as is (funny considering how the game was originally atmos sim)

@DoctoreMad
Copy link

Why don't we just spawn that atmos engineer's firelock hologram on areas where tiny fans should be
I mean its what I would do if I was NT Prez

@optimumtact
Copy link
Member

@Bawhoppen I take issue with "our atmos is fucked completely"

it is not, and it runs very well for what we're doing with it and the platform it's on

@Kromgar
Copy link
Contributor

Kromgar commented Jun 28, 2017

I think having a tiny fan at toxins is great. It's so annoying when you try to launch a bomb and it gets pulled out by space wind instead and gets stuck

@imtakingabreakdontatme
Copy link
Contributor

Mass drivers and the mining shuttle should have a fan on every map yeah

@Ergovisavi
Copy link
Contributor

It's a let's replace something that exists to make gameplay more smooth with something that exists to make gameplay more annoying, instead of just making the effect more believable episode

Stop. Please just make the tiny fan something more sci-fi and fitting instead of going out of your way to annoy people.

@imtakingabreakdontatme
Copy link
Contributor

The airlock cycle is frankly a pain in the ass when trying to let people into/out of the brig yeah

Copy link
Member

@Cyberboss Cyberboss left a comment

Choose a reason for hiding this comment

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

We have tons of methods for controlling atmos, tiny fans need to go

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Removal This was too fun, too fun! I'm turning this feature around
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet