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

Adds Freon to the game #20188

Merged
merged 18 commits into from
Aug 30, 2016
Merged

Adds Freon to the game #20188

merged 18 commits into from
Aug 30, 2016

Conversation

Iamgoofball
Copy link
Contributor

🆑 Iamgoofball
add: Adds a new experimental gas to the game. Check it out!
/:cl:

Starts in Toxins and Atmospherics roundstart on Meta and Box.

oyfbpg2

@@ -51,6 +51,13 @@ var/list/hardcoded_gases = list("o2","n2","co2","plasma") //the main four gases,
gas_overlay = "plasma"
moles_visible = MOLES_PLASMA_VISIBLE

/datum/gas/freon
id = "freon"
specific_heat = 1
Copy link
Contributor

Choose a reason for hiding this comment

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

Setting this to 1 is a terrible idea. Make it around 2000 and just have the gas start cold.

Copy link
Member

Choose a reason for hiding this comment

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

correct, I don't think you understand how specific_heat works. This isn't a temperature, but rather how well it stores temperature or lack of temperature.

With your current values here, 1 canister of o2 at room temperature could heat up 20 canisters of freon.

Set this to higher for freon, 2389 to be exact.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok

@WJohn WJohn added Atmos Feature Exposes new bugs in interesting ways FUCK Only two things are infinite, the universe and human stupidity, and I'm not sure about the former labels Aug 28, 2016
if(!IT.is_frozen)
make_frozen_visual(IT)
MakeSlippery(TURF_WET_PERMAFROST)
air.temperature = 2.7
Copy link
Contributor

Choose a reason for hiding this comment

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

This is so wrong, so very wrong. Just have the gas have a high specific heat and start out cold. Otherwise even the tiniest trace of this will keep a room cold forever unless you space the entire room and rebuild it. A station flood of this would be an instant shuttle call.


// Used to make the frozen item visuals for Freon.

/proc/make_frozen_visual(var/obj/F)
Copy link
Contributor

Choose a reason for hiding this comment

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

/obj/proc/make_frozen_visual()

Also there's probably a better file to put this in.

@ma44
Copy link
Contributor

ma44 commented Aug 28, 2016

waiting for AI to flood the station with this and plasma after the scientist uploads a free form law

@PKPenguin321
Copy link
Contributor

This looks really cool, wow 👍

@kyrahabattoir
Copy link
Contributor

So i'm the only one who would rather have this effect added to the standard gas system instead of making a meme gas?

It's not like we aren't simulating cold already right?

@Jalleo
Copy link
Contributor

Jalleo commented Aug 28, 2016

lets wait until making it a general thing until all the balance changes of standard stuff needs doing after a few large abuses of this system is done (antag or otherwise).

@kyrahabattoir
Copy link
Contributor

Like that's ever going to happen (the general thing part).

@as334
Copy link
Contributor

as334 commented Aug 28, 2016

I would actually also prefer it to be a general thing, but having the effect tied to a specific gas is currently the best way to try it out for balance and then see what needs to be tweaked.

@Iamgoofball
Copy link
Contributor Author

Also I like unique gases instead of jack of all trades

F.name = "frozen [temp_name]"
F.is_frozen = TRUE
return
return
Copy link
Contributor

Choose a reason for hiding this comment

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

return

@Anonmare
Copy link
Contributor

This might be a nice gas if you could carefully release small amounts of it to cool down areas that have been overheated.

@silicons
Copy link
Contributor

This seems kinda overpowered...

@Iamgoofball
Copy link
Contributor Author

Next commit is gonna change how the slide on the ice works.

@PKPenguin321
Copy link
Contributor

is this a port because if not this is actually really really creative and cool goofball like this is actually really neat

@Supermichael777
Copy link
Contributor

custom flamers
this could be fun

@Supermichael777
Copy link
Contributor

wait to be clear i can freez people solid right?

@Iamgoofball
Copy link
Contributor Author

@PKPenguin321 this is original content donut steel

except the sprites i borrowed those


// Used to make the frozen item visuals for Freon.

/obj/proc/make_frozen_visual(var/obj/F)
Copy link
Contributor

@RemieRichards RemieRichards Aug 28, 2016

Choose a reason for hiding this comment

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

should really drop the _visual since it actually sets the is_frozen var, making it REALLY frozen.
also why on earth does this take an argument and not just use src? (Edit: this last bit seems to be because you misunderstood @phil235's comment)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

pre refactor leftovers

@AKlimentyev
Copy link

@Iamgoofball talk to @MrStonedOne about that. I had issues with gulp not installing and he found some alternate way to install that worked.

@Razharas
Copy link
Contributor

  1. If its actually a layer of ice it should protect the dude inside from shit outside, this is both more consistent and makes it more than a stun-with-different-visuals
  2. Find a better sprite for iced status effect, the cube looks like 8bit snow level and feels really cheap, especially concidering the attenmpts to switch to x64

@Iamgoofball
Copy link
Contributor Author

  1. maybe after this PR is merged and I add icecubing to cryostal
  2. no

@optimumtact optimumtact merged commit 634662d into tgstation:master Aug 30, 2016
@Lexorion
Copy link
Contributor

Y E S

@Incoming5643
Copy link
Contributor

DO
NOT
MERGE

@optimumtact
Copy link
Member

The do not merge is stale it was applied 2 days ago.

@Incoming5643
Copy link
Contributor

Oranges I'm starting to think you live in some sort of alternate reality where the flow of time is an order of magnitude faster than it is for the rest of us.

@Iamgoofball
Copy link
Contributor Author

@Incoming5643 it was also applied in reference to the lag, which was fixed with caching and changes to how the gas worked

@ChangelingRain
Copy link
Contributor

And, uh, the animation issue still exists?

@Iamgoofball
Copy link
Contributor Author

@ChangelingRain no concievable way to fix it, i'm working on a BYOND report right now actually

@imtakingabreakdontatme
Copy link
Contributor

The balance issues also still exist but oh well

@imtakingabreakdontatme
Copy link
Contributor

So does the "teleport back and forth because of atmos/slipping interaction and get stuck until you die" bug

@YakumoChen
Copy link
Contributor

Can we not leave oranges to merge this when there are a ton of people who don't want it and balance issues still exist

@imtakingabreakdontatme
Copy link
Contributor

Also it has no purpose at all in regards to any civilian use as far as I'm aware

@imtakingabreakdontatme
Copy link
Contributor

And made space even more binary "if you don't have a suit you're fucking dead within seconds" because now space stuns you

@YakumoChen
Copy link
Contributor

corpses become ice cubes in space and it looks disgusting visually.

@Saegrimr
Copy link

Also it has no purpose at all in regards to any civilian use as far as I'm aware

If it could be used to make a safe Supermatter, i'd be all for it.

@Iamgoofball
Copy link
Contributor Author

@Saegrimr it's cold as fuck so yeah

@PKPenguin321
Copy link
Contributor

Why would you ever use gas to cool supermatter if you are literally in the infinite coldness of space

@Alek2ander
Copy link
Contributor

Ice cubes don't really make sense. They're fun, I admit, but they shouldn't happen every time something gets cold.
Possibly only icecube people when actually exposed to freezegas, or add some sort of water stacks (Magicka freezing comes to mind)

@Iamgoofball
Copy link
Contributor Author

@Alek2ander #20233

@Saegrimr
Copy link

@PKPenguin321
Because it just doesn't work that way.

@nullbear
Copy link
Contributor

iirc Supermatter only processes when on a non-space simulated turf.

@imtakingabreakdontatme
Copy link
Contributor

You can just RCD the floor next to it though

@Saegrimr
Copy link

0/10 would not recommend for supermatter http://i.imgur.com/ICU8gOx.png

@NocturnalQuill
Copy link

Items that are extremely hot like supermatter shards should be immune to freon freezing.

@AKlimentyev
Copy link

the frozen supermatter shard has reached the emergency point.

Absolutely amazing.

@@ -719,6 +719,10 @@ var/next_mob_id = 0
layer = initial(layer)
update_transform()
update_action_buttons_icon()
if(istype(src, /mob/living))
Copy link
Contributor

Choose a reason for hiding this comment

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

😢

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Do Not Merge You must have really upset someone Feature Exposes new bugs in interesting ways FUCK Only two things are infinite, the universe and human stupidity, and I'm not sure about the former
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet