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

nerfs force glove damage amplification to melee weapons. #5265

Closed
wants to merge 7 commits into from
Closed

nerfs force glove damage amplification to melee weapons. #5265

wants to merge 7 commits into from

Conversation

scheveningensun
Copy link
Contributor

@scheveningensun scheveningensun commented Sep 10, 2018

With the amount of high base damage melee weapons that exist in the game now, the game and the state of roleplay would benefit more with this trim back of the increasing power creep stemming from the combination potential of force gloves and the highest damaging weapons.

It's a roleplay server, anyway, insta-dying shouldn't become common.

Edit: After some feedback, the damage amp was reduced instead.

@Nerev4r
Copy link
Contributor

Nerev4r commented Sep 10, 2018

I don't believe being this hamhanded is good, instead, just make force gloves more expensive.

@scheveningensun
Copy link
Contributor Author

https://forums.aurorastation.org/viewtopic.php?f=18&t=11881&start=10

feedback thread for posterity.

@Arrow768 Arrow768 added the Feedback Required The PR requires a feedback topic and feedback on the forum. label Sep 11, 2018
if(X.gloves && istype(X.gloves,/obj/item/clothing/gloves/force))
var/obj/item/clothing/gloves/force/G = X.gloves
power *= G.amplification
//if(istype(user, /mob/living/carbon/human)) //Removed.
Copy link
Member

Choose a reason for hiding this comment

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

It should be removed properly and not just commented out.
If someone wants to restore it, they can just look up how it was previously (thanks to git)

@scheveningensun
Copy link
Contributor Author

!review

Copy link
Member

@Arrow768 Arrow768 left a comment

Choose a reason for hiding this comment

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

Technically alright. Still needs feedback.

@scheveningensun scheveningensun changed the title Removes force glove damage amplification to melee weapons. nerfs force glove damage amplification to melee weapons. Sep 12, 2018
@@ -16,7 +16,7 @@

/datum/uplink_item/item/visible_weapons/forcegloves
name = "Force Gloves"
item_cost = 8
item_cost = 5
Copy link
Contributor

Choose a reason for hiding this comment

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

Hmm... I would say that if we kept the damage multiplier at 2.5, the cost should be at least 10-14 TC. But since we reduce damage multiplier hence we need to reduce price as if we bumped it to 10-14 TC. So 8-10 is still reasonable price for 1.5 damage multiplier, what do you think?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The thing I'm taking into consideration is removing a substantial amount of burst damage from some melee weapons (which might even be considered necessary since we've plenty of melee weapons that hit between the 15-30 force benchmarks of being considered strong).

Having a limb taken off in a single blow is not very fun, especially since weapons with 25-30 force get amplified to an amount that makes them cut off the lower torso from people's body. If you lose your lower torso you die instantly, and the lower torso is easier to hit than the head.

I can probably tune the cost again, and maybe settle the damage mult to 1.75, so that force gloves are still good but not overtuned.

Copy link
Contributor

Choose a reason for hiding this comment

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

My point was that the price you set for 1.5 was too low for benefit of it, I think it should have been 8-10 for 1.5. If you are gonna make it 1.75 then it should be 9-12 TC.

@Arrow768 Arrow768 added this to the Update November 2018 milestone Oct 7, 2018
@scheveningensun
Copy link
Contributor Author

bump

@BotBOREALIS BotBOREALIS removed this from the Update November 2018 milestone Oct 16, 2018
@scheveningensun scheveningensun deleted the gloves branch December 1, 2018 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feedback Required The PR requires a feedback topic and feedback on the forum. Review Required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants