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

Hyperzine prevents Diona regeneration. Hyperzine no longer speeds up Diona as much as it used to. #14721

Merged
merged 5 commits into from May 14, 2017

Conversation

Anton-Kr
Copy link
Contributor

@Anton-Kr Anton-Kr commented Apr 23, 2017

Tired of green fuckers running to your chem station every round begging for their pills to make them even more OP? Worry no longer.

Changelog

🆑

  • tweak: Hyperizine prevents Diona health regeneration
  • tweak: Hyperizine speedups for Diona are nowhere near as powerful as they were before.

@ultraplen
Copy link
Contributor

A fair nerf. Possibly a bit harsh.

@BobdaBiscuit
Copy link
Contributor

If only you knew what kind of hell you are inviting in by doing this.

@Kurfursten
Copy link
Collaborator

With the current numbers, this will crit a Diona in 13 seconds, consuming 0.39u Hyperzine

The low lethal dosage should be taken into account when considering splash/smoke mechanics

@PJB3005
Copy link
Contributor

PJB3005 commented Apr 23, 2017

Yeah the damage is WAY over the top.

Make it more manageable but still not something you want to consistently use.

@PJB3005 PJB3005 added the Balance Potentially going to upset people as it changes balance of the game. label Apr 23, 2017

if(ishuman(M))
var/mob/living/carbon/human/H = M
if(H.species.name == "Diona")
Copy link
Contributor

Choose a reason for hiding this comment

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

Use isdiona(H)

@Fruktlimpa
Copy link
Contributor

Dionae is a pretty shit race lets not lie. The massive slowdown would make them extremely annoying to play which is why everyone chugs hyperzine in the first place.
Instead of making hyperzine a death sentence or forcing them to chug other pills at the same rate. Isn't it possible to make them slightly slower than a human by consuming hyperzine?
I realize it isn't ideal because it would still force them to chug hyperzine but at least then they wouldn't be speedy trees of no brute damage.

@Anton-Kr
Copy link
Contributor Author

Im going to make the damage dealt a lot more reasonable (and potentially counteracted with chems) and also make the speed up to not be as great as it is for other species.

@Anton-Kr
Copy link
Contributor Author

Will also try to make regeneration not work for diona while on hyperzine in which case Ill probably reduce damage dealt even more so

@BobdaBiscuit
Copy link
Contributor

IMO, if you make hyperzine not make dionas sanic, and then disable their regen while under it's effects the damage is unneeded.

@Ephx1
Copy link

Ephx1 commented Apr 23, 2017

It begins!

@Blithering
Copy link
Contributor

Disabling the regen when hyperzine is present as @BobdaBiscuit suggested seems like a fair tradeoff to me, not sure how feasible that is code wise though

@BobdaBiscuit
Copy link
Contributor

It's perfectly possible, I had a look through the code for em and it's just a isdiona() + has hyperzine test and we're gucci

@Kurfursten
Copy link
Collaborator

I think disabling the regeneration is an interesting idea, because combat is when you would most want regeneration AND movement speed, and it's not trivial for most jobs to remove the hyperzine when combat is over unless you're carrying a pAI to smoke it out.

Overall, I appreciate its simplicity and balance as an idea.

@Anton-Kr
Copy link
Contributor Author

Alright, so no damage dealt to diona, but no regen, and speed buff is nerfed.

@Exxion
Copy link
Member

Exxion commented Apr 23, 2017

If it's less effective AND prevents regeneration, is the damage really necessary?

@Anton-Kr
Copy link
Contributor Author

@PJB3005 Let me know if there are any other issues.

@Anton-Kr
Copy link
Contributor Author

@Exxion It isn't. I forgot to remove it in last commit.

@Anton-Kr Anton-Kr changed the title Makes hyperzine poisonous for diona. Hyperzine prevents Diona regeneration. Hyperzine no longer speeds up Diona as much as it used to. Apr 23, 2017
@gbasood
Copy link
Contributor

gbasood commented Apr 24, 2017

Dionae is a pretty shit race lets not lie.

Yeah man regenerating through anything but burn damage
So shit

@Exxion
Copy link
Member

Exxion commented Apr 24, 2017

And taking enough burn damage to be critted in two (2) standard laser shots

@Exxion
Copy link
Member

Exxion commented Apr 24, 2017

Consider these options:

  • Git Gud

@gbasood
Copy link
Contributor

gbasood commented Apr 24, 2017

I'd rather make a balance change that's overkill and then find a halfway point afterwards than fiddle around with multiple PRs that are "too harsh man" according to a small subset of players

@ghost
Copy link

ghost commented Apr 24, 2017

just make hyperzine heat up bodies enough to deal burn if you overdose

@Fruktlimpa
Copy link
Contributor

@gbasood

Yeah man regenerating through anything but burn damage
So shit

I meant more in a sense that they are a stupid meme race that literally cannot function without hyperzine. Anyone who would willingly play Dionae without so much as a drop of hyperzine is insane.

@ghost
Copy link

ghost commented Apr 24, 2017

I think this PR is bad because it changes the two main diona things, which is being slow and having high regen. Diona already suck, even with non nerfed hyperzine. Here is my ideas to fix diona, and make them more like tree people they should be.

slightly faster base movement speed (still slower than human)
unaffected by zas
chemicals don't work on them

No zas, can't be chloral or sleepypen meme'd but they're stuck being sorta slow forever because they can't eat hyperzine.

@Shadowmech88
Copy link
Contributor

I keep seeing people claim that dionae are OP.

Now, I haven't played in a while, so I'm honestly asking: are there a significant number of players rolling dionae? Something like 30%-40% at least?
Because that's what I would expect to happen if we had a race that was truly overpowered, that a significant number of people would be playing it for the advantage it offered.

@BobdaBiscuit
Copy link
Contributor

Actually we had a week were the majority of sec teams were all diona, is was a pretty funny meme.

@SonixApache
Copy link
Contributor

SonixApache commented Apr 24, 2017

if you're going to take away their regen you might as well make them as fast as humans
level the playing field rather than home advantage

and while we're at it, antitox ODing when?

@Anton-Kr
Copy link
Contributor Author

Regen is still there but only when they have no hyperzine in system

@SonixApache
Copy link
Contributor

Yes, that's the point. If they don't have their bullshit regeneration anymore, why make them slower still than humans on drugs?

@FudgePucker
Copy link
Contributor

Number one murder bone weapon is egun.
Zap and laser to death.
Number one lethal option security has is egun.
Zap and laser to death.
Everybody has a toolbelt with a welder.
Stuns are king burn damage is second.

Hell they can't survive polyacid smoke grenades. How are they super OP again?

I like Solarious' idea. They are so slow it takes half a round to cross the station, you take hyperzine or roleplay a tree. A minor speed nerf instead of a super major "oh god how can it be possible to move this slow" one. Nerf regen so you can kill one in full light with a esword and then we have balance.

@Blithering
Copy link
Contributor

Combat isn't the only factor when considering a race's strengths. Remember they don't need to breathe or eat, are unaffected by pressure, heal from radiation, can shrug off ZAS and ignore overdose limits on meds. Surviving as a diona is a lot simpler because you can pretty much ignore everything that isn't fire.

@Exxion
Copy link
Member

Exxion commented Apr 24, 2017

At the cost of walking at roughly the same speed as a slime mold.
Which is why hyperzine is too good for them, which is why this PR exists

Also, all of you fucks saying that dionae are unplayably slow need to either get more patient or stop playing dionae

@Intigracy
Copy link
Contributor

Clearly we should just make dionae unable to move.

@Shadowmech88
Copy link
Contributor

They are trees, after all. Trees don't move.

@Intigracy
Copy link
Contributor

wait sorry wrong meme

the obvious solution is to remove skeletons

@Intigracy Intigracy dismissed PJB3005’s stale review April 25, 2017 10:00

Did what he was told

@DamianX
Copy link
Contributor

DamianX commented Apr 25, 2017

Fix the changelog already

Copy link
Contributor

@ComicIronic ComicIronic left a comment

Choose a reason for hiding this comment

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

Needs fixing.

@@ -76,6 +76,8 @@
if(reagents.has_reagent(HYPERZINE))
if(dna.mutantrace == "slime")
tally *= 2
if(isdiona(src))
Copy link
Contributor

Choose a reason for hiding this comment

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

Use else if.

@@ -42,7 +42,7 @@
if(species.flags & IS_PLANT)
if(nutrition > 500)
nutrition = 500
if(light_amount >= 3) //If there's enough light, heal
if(light_amount >= 3 & !reagents.has_reagent(HYPERZINE)) //If there's enough light, and you do not have hyperzine in body, heal
Copy link
Contributor

Choose a reason for hiding this comment

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

Use &&, & is a bitwise operation.

@Anton-Kr
Copy link
Contributor Author

@ComicIronic done

@BobdaBiscuit
Copy link
Contributor

How fast is it ingame compared to the human base speed? Cause the tally boost seems hella lower.

@Anton-Kr
Copy link
Contributor Author

@BobdaBiscuit Human slowdown by default is 0. Diona is 7 without hyperzine, and 3 with, so somewhat slower then humans still.

@BobdaBiscuit
Copy link
Contributor

Eh if it's still too slow I suppose another pr can be made to tweak it.

@Anton-Kr
Copy link
Contributor Author

MERGE WHEN

@BobdaBiscuit
Copy link
Contributor

BobdaBiscuit commented May 14, 2017

>"fair" diona nerf that most people agree with
>still not merged nearly a month later

REALLY MAKES YOU THINK

@Intigracy
Copy link
Contributor

If this needs adjusting we can do so.

@Intigracy Intigracy merged commit 1b1ca39 into vgstation-coders:Bleeding-Edge May 14, 2017
ihadtoregisterforthis pushed a commit to ihadtoregisterforthis/fork4 that referenced this pull request Jul 3, 2017
…Diona as much as it used to. (vgstation-coders#14721)

* 'Done'

* fixes

* Forgot to remove damage

* Fix 2
d3athrow pushed a commit that referenced this pull request Jul 4, 2017
…Diona as much as it used to. (#14721)

* 'Done'

* fixes

* Forgot to remove damage

* Fix 2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance Potentially going to upset people as it changes balance of the game.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet