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

Alternative To Removing Kudzu Manufacturing (#30080) #30084

Merged
merged 7 commits into from Aug 23, 2017
Merged

Alternative To Removing Kudzu Manufacturing (#30080) #30084

merged 7 commits into from Aug 23, 2017

Conversation

ExcessiveUseOfCobblestone
Copy link
Contributor

馃啈 Cobby
balance: Planting kudzu now has a short delay with a visible message to users around you. Hit-N-Run planting is no longer possible.
del: kudzu bluespace mutation and spacewalk mutation are removed.
/:cl:

closes #30080

This nerf strikes at 2 major problems, Hit-N-Run planting and kudzu going through walls/space, while still making it a station destroyer if done properly and with more effort than left clicking seeds while inhand.

If this is still unmanageable I'll lower the spread by 25% but I think this will be fine tbh.

Oh I did test this but my github local branch is out of date due to not using this computer in forever and so I had to transfer it from desktop to online via web editor. pls no bully gunhog

@tgstation-server tgstation-server added Balance Changes to functionality that modifies how effective certain methods are at powergaming Removal This was too fun, too fun! I'm turning this feature around labels Aug 22, 2017
@optimumtact optimumtact self-assigned this Aug 22, 2017
@Kromgar
Copy link
Contributor

Kromgar commented Aug 22, 2017

That thorny black kudzu should still break down doors shouldn't it?

@Tokiko1
Copy link
Contributor

Tokiko1 commented Aug 22, 2017

If I remember correctly there is a mutation that ex_act's stuff too, which means that it breaks doors/walls. Maybe.

@lordpidey
Copy link
Contributor

I've seen kudzu break down walls somehow, it's probably the ex_act mutation @Tokiko1 mentioned, but let's double check.

@DoctoreMad
Copy link

Explosive kudzu mutations are horrible machines created by Moloch form his throne in hell and sent towards space to destroy the world of spessmen. I'm surprised that bluespace is being removed but that meme isn't.

Is it because none of the coders died to a ex-act kudzu vine explosion yet?

@CPTANT
Copy link
Contributor

CPTANT commented Aug 22, 2017

These changes are good and sufficient nerfs for now.

Lets not over-nerf something for a change.

@XDTM
Copy link
Contributor

XDTM commented Aug 22, 2017

Aggressive spreading calls ex_act, along with esplosive kudzu. The latter should be removed mostly because there's no way to really fight it.

@Grazyn
Copy link

Grazyn commented Aug 22, 2017

I've seen explosive kudzu level entire departments in a couple seconds with its chain reaction. Like literally leveled, everything in a huge area destroyed leaving only ruined plating behind. Imagine how an area looks after a massive blob is killed, that was it.

@jughu
Copy link
Contributor

jughu commented Aug 22, 2017

Keep the growing trough doors kudzu just add a delay to planting

@Kromgar
Copy link
Contributor

Kromgar commented Aug 22, 2017

Maybe instead of ex_act kudzu make it act like blob tiles that just attack things around them. Could that be done?

@Cyberboss
Copy link
Member

Remove the doors thing, don't nerf it into the ground

@ExcessiveUseOfCobblestone
Copy link
Contributor Author

I removed spacewalking kudzu too because lol finding it in space.

However oddly enough that airlock check was only in the space walk mutation yet applied to normal kudzu. how weird.

Now I wait patiently for my branch to update while on campus wifi to test this.

@ExcessiveUseOfCobblestone
Copy link
Contributor Author

Explosive kudzu mutations are horrible machines created by Moloch form his throne in hell and sent towards space to destroy the world of spessmen. I'm surprised that bluespace is being removed but that meme isn't.
Is it because none of the coders died to a ex-act kudzu vine explosion yet?

Well this PR was to attack the problems with kudzu found in PR that's referenced in the title. Not a single post on said PR mentioned explosives, but several people suggested removing bluespace.

With that said, you can see how easy it is to remove a mutation, and this PR shows you the exact file to go to. If it bothers you that much, you're more than welcome to do your own PR removing that mutation as well.

@XDTM
Copy link
Contributor

XDTM commented Aug 22, 2017

Fair enough, i usually ask for additions because that way there's no need to wait for another 24 hours to merge.

Either here or by someone else though, explosive kudzu should really get a different trigger or removed altogether, because of the lack of counters to it.

@ExcessiveUseOfCobblestone
Copy link
Contributor Author

I personally would like to see how this goes before further nerfing it further, hence my hesitation.

@Arianya
Copy link
Contributor

Arianya commented Aug 22, 2017

Gonna mirror comment here that kudzu need to have their "able to live in space" feature nerfed or removed entirely. Anecdotally it causes MC issues, and less anecdotally it is a humongous pain in the ass to deal with since it expands out till the edge of the z-level with no real barrier.

@ExcessiveUseOfCobblestone
Copy link
Contributor Author

@Arianya it removes the mutation entirely

@CPTANT
Copy link
Contributor

CPTANT commented Aug 22, 2017

Kudzu only growing in space adjacent to the station would be good.

plant(user)
to_chat(user, "<span class='notice'>You plant the kudzu. You monster.</span>")
user.visible_message("<span class='danger'>[user] begins throwing seeds on the ground...</span>")
if(do_after(user, 50, needhand = 1, target = user, progress = 1))
Copy link
Member

Choose a reason for hiding this comment

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

needhand = TRUE, target = get_turf(user), progress = TRUE

@Cyberboss Cyberboss merged commit 9055e89 into tgstation:master Aug 23, 2017
ghost referenced this pull request in HippieStation/HippieStationdeprecated2020 Aug 23, 2017
* Update kudzu.dm

* Update spacevine.dm

* Update spacevine.dm

* b

* Removes Airlock nerf [why did this work anywho]

* Removes Spacewalking kudzu

* Eh, I'll do this myself
Cyberboss added a commit to Cyberboss/tgstation that referenced this pull request Aug 23, 2017
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 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