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

Reverts shrapnel damage changes from #7581 #8613

Merged
merged 1 commit into from
Mar 6, 2025

Conversation

Mister-moon1
Copy link
Contributor

@Mister-moon1 Mister-moon1 commented Feb 28, 2025

About the pull request

Title

Explain why it's good for the game

Changes were untested, over-tuned and a net negative for the game, numerous interaction changes such as all shrapnel from a NON OT claymore can deal upwards of 400 damage under the circumstances of all shrapnel hitting, possibly killing a majority of t2s without a chance to dispute said damage, this among every single other thing that uses shrapnel, breach charges, grenades, etc, make it unfun and quite frankly just stupid to fight.

Testing Photographs and Procedure

https://streamable.com/kplvey

https://streamable.com/xkt19k

Changelog

🆑
balance: reverted all direct shrapnel buffs from #7581
/:cl:

@cmss13-ci cmss13-ci bot added the Balance You need to be a professional veteran game maintainer to comprehend what is being done here. label Feb 28, 2025
@Mister-moon1
Copy link
Contributor Author

correction it was tested over the span of around 14 hours, mostly during lowpop, regardless I would not call it nearly adequate for a flat 60% buff to something as abundant as shrapnel

@iloveloopers
Copy link
Contributor

It is very likely you also were double-hit from the explosion in the second clip as you were moving downward.

@private-tristan
Copy link
Contributor

private-tristan commented Feb 28, 2025

It is very likely you also were double-hit from the explosion in the second clip as you were moving downward.

nope, #8470 was TM'd during that clip (you can see the explosion being instant).

@iloveloopers
Copy link
Contributor

nope, #8470 was TM'd during that clip (you can see the explosion being instant).

Anectodal evidende but it certainly feels like it is possible to be double hit with "instant" cellauto explosions and it seems to happen more frequently.

@Git-Nivrak Git-Nivrak added this pull request to the merge queue Mar 6, 2025
Merged via the queue into cmss13-devs:master with commit 79c35e7 Mar 6, 2025
28 checks passed
cmss13-ci bot added a commit that referenced this pull request Mar 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance You need to be a professional veteran game maintainer to comprehend what is being done here.
Projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

4 participants