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

[READY] Explosive Tesla, Take 2 #20083

Merged
merged 7 commits into from Aug 28, 2016

Conversation

PKPenguin321
Copy link
Contributor

@PKPenguin321 PKPenguin321 commented Aug 24, 2016

Video (a bit outdated): https://www.youtube.com/watch?v=f_CrgMbk4t4
Remake of #17501

Todo:

  • Make tesla itself bomb immune since it will delete itself in some cases
  • Make an upgrade for cameras and airlocks light tubes that tesla-proofs them to prevent tesla from breaking its own containment
  • Do a testmerge of this to get a feel for how laggy it is and adjust explosion values and probabilities accordingly

Maps Tested On:

  • Boxstation: Containment area can successfully hold a tesla ball with the equipment given at roundstart. Lights occasionally blow out, but it's inconsequential.
  • Metastation: Also works.
  • Birdboat/ministation: Doesn't even have a singulo containment area so no issue there
  • Dreamstation: ✅
  • EfficiencyStation ✅ Lights occasionally blow out, but it's inconsequential.

Changes made:

  • tesla_act for machinery in general now often calls minibomb-sized explosions (which lack smoke and admin logging to reduce lag and spam, respectively)
  • explosion proc now has an argument that lets you determine if you want your explosion to make smoke or not. Defaults to 1
  • The effect_system itself for explosions is split into two paths, one that makes smoke and one that doesn't (should be inconsequential)
  • Makes light tubes and their variants spew fire on tesla_act instead of a full explosion (to keep tesla from breaking itself out of containment)
  • Makes cameras with EMP immunity upgrade also tesla immune
  • Makes the tesla itself bomb immune
  • Refactored tesla grilles. They no longer shoot a real tesla beam, just one that looks exactly like a tesla beam and will ONLY target carbon mobs (meaning no explosions). As a consequence, they will only ever target 1 person at a time, meaning you can't kill 40 people with one zap anymore.
  • Made the gravity generator get properly destroyed by the tesla so that it doesn't just repeatedly explode over and over.
  • Made the nuke do the same as the gravity generator. As a consequence, nukes can now be "disarmed" by the tesla (in the same way BoH bombing a nuke disarms it).

Other notes:

  • I've been able to make the tesla successfully without it breaking loose on it's own. on Boxstation, but other maps are yet to be tested. It's been fully tested and proper engine setup should never lead to a tesloose.
  • Airlocks do not need to be bomb-proofed as far as I can tell thanks to grounding rods being right next to them. (at least on Boxstation)

The goal here is to make the tesla about as destructive as the singularity is, instead of EMPing and occasionally deleting some machines.

🆑 PKPenguin321
add: The tesla is now much more dangerous, and can cause electronics to explode violently.
/:cl:

@WJohn WJohn added Balance Changes to functionality that modifies how effective certain methods are at powergaming Feature Exposes new bugs in interesting ways labels Aug 24, 2016
@WJohn
Copy link
Contributor

WJohn commented Aug 24, 2016

Holy shit. I've been asking for this for a while. 👍 x 💯

@Core0verload
Copy link
Contributor

👎 Tesla is already an unkillable round ender. Make it decay if you really want this.

@PKPenguin321
Copy link
Contributor Author

If it's gonna be an unkillable round ender, might as well make it go all-out instead of just EMPing things, right? More fun that way

@AKlimentyev
Copy link

@PKPenguin321 It deletes shit by calling ex_act on it as well.

@PKPenguin321
Copy link
Contributor Author

Yep! The singularity also deletes shit, but it just does it directly, so I guess you could say this is a step down.

- makes tesla bomb immune so it wont destroy itself
- makes lights spew fire when they explode instead of the regular explosion
- makes cameras with EMP immunity also have tesla immunity
tesla can now keep itself fully contained (at least on box)
@PKPenguin321
Copy link
Contributor Author

PKPenguin321 commented Aug 25, 2016

Alright then, all that's left is to testmerge and tweak values. I'm starting with explosions being the most likely thing and them being minibomb-strong, which is pretty much the maximum I'll go with this. If it's laggy (which is likely) I will likely do one or more of the following:

  • Make explosions not the most likely option
  • Make multiple probabilities with weak explosions being the most common and strong being the least
  • Lower the tesla's range a little bit so it can't proc as many explosions at once

@ShadowDeath6
Copy link

don't minibombs instant crit if you're point blanked with them?

now we get double instacrit tesla action, with 100% less counter memes

@PKPenguin321
Copy link
Contributor Author

The singularity insta_gibs,_ so you're still better off.

@TechnoAlchemisto
Copy link
Contributor

This is a good thing, a loosed engine is supposed to be ridiculously chaotic and dangerous. Lethality memes.

@Cheridan
Copy link
Contributor

Cheridan commented Aug 25, 2016

Singularity instagibs... if you manage to fall into it, which is surprisingly avoidable.
Tesla just pops onto your screen and instakills you and everyone you love.

The difference between gibbing and killing in this situation is pretty meaningless since if the tesla is loose, the shuttle is coming. You're probably not getting cloned before the round ends.

@PKPenguin321
Copy link
Contributor Author

The difference between gibbing and killing in this situation is pretty meaningless since if the tesla is loose, the shuttle is coming. You're probably not getting cloned before the round ends.

Yep, and the difference between machines getting emagged and blown up is pretty meaningless if the shuttle is coming, so why not have the more exciting option?

@silicons
Copy link
Contributor

silicons commented Aug 26, 2016

I rescind my hate. 10/10, MAKE IT STRONGER!!

@Ninja137
Copy link

It's beautiful. Make it lose charge slower, though.

@Grandsteel
Copy link

The perfect concept, actually makes the tesla coil fun to watch

@PKPenguin321
Copy link
Contributor Author

PKPenguin321 commented Aug 26, 2016

Do you rike it
Too strong: 1
It's perfect: 4
MAKE IT STRONGER: 17
Did not vote: 42

Testmerge observations:

  • No lag!!!!!! It's perfect! 😄
  • Make mechs asplode Edit: Changed my mind about this one, it's essentially just like hardsuit immunity so I figure I'll just leave it be.
  • Maybe make nukes and grav gen get deleted instead of exploding over and over
  • Maybe remove hardsuit immunity from tesla? (Will save for another PR in any case)
  • Players in deadchat fucking loved it, a few said to lower the range but overall feedback was very high in praise
  • You could hear the tesla before it got to you and started bombing, so that's good

@AKlimentyev
Copy link

AKlimentyev commented Aug 26, 2016

One of the grounding rods got moved from its position and the tesla hit an engineer, gibbed them, and blew a huge hole in engineering.
It then hit another spot. I think it was the collectors, and cut the wire to emitters.
There's going to be a fuckton more teslalooses without engineering even fucking up.
Explosions are such a terrible idea, since the aoe means its super easy for it to hit a wire and escape.

@TechnoAlchemisto
Copy link
Contributor

I wouldn't remove the tesla immunity considering everything around you explodes.

@Ninja137
Copy link

Ninja137 commented Aug 26, 2016

No. What happened was this:

-Set up engine
-[REDACTED] happens
-Panic, forget what I'm doing
-Continue setting up engine
-Accidentally put in the Singulo
-Turn on after hacking PA
-Singulo moves rods
-Ask admins to replace engine with tesla
-No grounding rods
-I'm standing adjacent to the eastern collector bank
-The very first shock explodes the entire side of the room and I'm immediately disintegrated, and practically jump out of my chair

Round Highlights of Tesla:

-Maxwell Gaylord's enfeebled right leg of elf slaying -11 has been violently dismembered!
-Bubba Squanto's lesser left arm of the plains -3 has been violently dismembered!
-Bubba Squanto's true right arm of elf slaying +12 has been violently dismembered!
-Nathaniel Saylor's inferior left arm of dexterity -4 has been violently dismembered!
-Tesla ripping through the entirety of Engi, Atmos, Med, Sci, Viro, and Escape in under two minutes after escaping

@PKPenguin321
Copy link
Contributor Author

I should also say that before this gets merged it needs to be tested on other maps. I know box can contain it, but I don't know about meta or any of the other maps. Map edits are likely needed (which sucks because I have no idea how to do them), it might be good to also put reinforced plating over the wires that lead to emitters, I don't know.

@Ninja137
Copy link

No man, it has to be able to escape. It can't natively escape, so it shouldn't be too massive an amount of shit to make it go kill everything.

@Iamgoofball
Copy link
Contributor

@Ninja137 The singulo also can't escape by itself it requires malpractice or sabotage

I really don't like this. It completely wrecks the design of the tesla.

@@ -466,10 +466,8 @@ Class Procs:
/obj/machinery/tesla_act(var/power)
..()
if(prob(85))
emp_act(2)
explosion(src.loc,1,2,4,flame_range = 2, adminlog = 0, smoke = 0)
Copy link
Contributor

Choose a reason for hiding this comment

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

Like I said before, this should be the LEAST common effect, not the most. This makes the tesla absolutely rip the station in half, like what the hell.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This makes the tesla absolutely rip the station in half

Rip the station in half?
You mean like the singularity?

Oh, good, working as intended.

Choose a reason for hiding this comment

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

didn't people not want this to be a copy paste of the singlo? I get boosting its devastation but making it explode stuff pretty much every time it zaps would be kind of lame

Copy link
Contributor

Choose a reason for hiding this comment

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

yeah this is not supposed to just be a copypasta of singularity

Copy link
Contributor

Choose a reason for hiding this comment

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

The tesla was supposed to be a lag free engine if I remember correctly

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is very nearly lag free during test-merges

@Grandsteel
Copy link

I cannot tell you my disappointment with old Tesla, when it was just a mass murdering machine IF it even moves around to people. Exploding consoles and other electronics manages to actually give it some radical damage. Also complaints about it's damage being too strong feel invalid, since you have to compare it to singularity, wherein Tesla is much less damage since explosions are only around machinery, which can be avoided. Also Tesla is still kinda mediocre to begin with since it tends to get stuck in certain areas, the only reason it absolutely fucked the station in the ass was because penguin set the Tesla to home in on people that prayed at a certain time.

@AKlimentyev
Copy link

@PKPenguin321 The explosions apply to the tesla zaps from throwing stuff at shocked grills.
Someone threw a toolbox and blew up the bridge.

@ShadowDeath6
Copy link

why were tesla grilles even added, they're almost an instakill even when they're low power

@PKPenguin321
Copy link
Contributor Author

@Iamgoofball I don't want to come off as rude, but the players in deadchat absolutely loved what I did to the tesla. It might go against your "design vision," but I heard time and time again that the tesla is now "much more fun to watch." You could ask almost anyone and they would agree that the current tesla we have right now is very, very boring, especially compared to the singularity.

This is just outright better and more popular/fun to the players, and there's no denying that.

@Iamgoofball
Copy link
Contributor

I guess. Just see if you can make the explosion visuals a little more electrified.

@PKPenguin321
Copy link
Contributor Author

In this commit:

  • Refactored tesla grilles. They no longer shoot a real tesla beam, just one that looks exactly like a tesla beam and will ONLY target carbon mobs (meaning no explosions).

@Iamgoofball
Copy link
Contributor

👎 no no no

they need to use proper tesla zap code, don't snowflake it

just disable the explosion for the tesla grille

@PKPenguin321
Copy link
Contributor Author

I tried like 40 different methods and spent two days on that, this was the best I could do. If you have an actual code-based method for removing the explosion for tesla grille, please please let me hear it.

@Iamgoofball
Copy link
Contributor

i'll look into it fam dont worry

@PKPenguin321
Copy link
Contributor Author

In this commit:

  • Made the gravity generator get properly destroyed by the tesla so that it doesn't just repeatedly explode over and over.
  • Made the nuke do the same as the gravity generator. As a consequence, nukes can now be "disarmed" by the tesla (in the same way BoH bombing a nuke disarms it).

@PKPenguin321 PKPenguin321 changed the title Explosive Tesla, Take 2 [READY] Explosive Tesla, Take 2 Aug 27, 2016
@PKPenguin321
Copy link
Contributor Author

@Jordie0608 @Razharas @optimumtact @ChangelingRain @RemieRichards @tkdrg @Dorsisdwarf @KorPhaeron
First person to merge this is confirmed cool

@RemieRichards
Copy link
Contributor

I'm already cool though.

@PKPenguin321
Copy link
Contributor Author

But are you confirmed cool

@RemieRichards
Copy link
Contributor

yes.

@PKPenguin321
Copy link
Contributor Author

shoot well could you still merge this please

@optimumtact
Copy link
Member

Time to make space great again.

@optimumtact optimumtact merged commit f800987 into tgstation:master Aug 28, 2016
@PKPenguin321 PKPenguin321 deleted the explosive-tesla branch August 28, 2016 00:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance Changes to functionality that modifies how effective certain methods are at powergaming Feature Exposes new bugs in interesting ways
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet