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

Berserker ravager low rage eviscerate tweaks #3992

Closed

Conversation

GoldenDarkness55
Copy link
Contributor

@GoldenDarkness55 GoldenDarkness55 commented Jul 25, 2023

About the pull request

Tweaks the damage numbers so the growth curve makes more sense and sets the minimum eviscerate delay at what was previously rage level 4 (5 is max) duration to prevent ravs from dying via locking themselves in place for no benefit through accidental key presses.

Explain why it's good for the game

Stunlocking yourself for no benefit as a t3 because you pressed the wrong button shouldnt be a thing, its extremely punishing for both new and skilled players alike for no real reason - no one ever intentionally uses the ability below the max rage level because it is pointless. Hopefully leads to less dumb deaths on one of the most skill reliant strains.

As for the damage part - raised it on lower levels as it was astronomically low and had massive spikes in growth rather than a linear curve. Mostly because it bothered me - I do not see any real use for it even with these changes, but maybe once in a blue moon someone will find a reason to go for it (it's still 1 range so balance impact is essentially null.)

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑
balance: Berserker eviscerate windup below rage lvl 4 set to level 4 values (0, 2 and 4 to 6)
balance: Berserker eviscerate damage raised on lower levels (5, 10, 25, 45 to 10, 25, 35, 50)
/:cl:

@github-actions
Copy link
Contributor

You currently have a negative Fix/Feature pull request delta of -5. Maintainers may close this PR at will. Fixing issues or improving the codebase will improve this score.

@github-actions github-actions bot added the Balance You need to be a professional veteran game maintainer to comprehend what is being done here. label Jul 25, 2023
@ihatethisengine
Copy link
Contributor

Why don't remove freezing on place so you can change your mind and cancel the ability?

@GoldenDarkness55
Copy link
Contributor Author

GoldenDarkness55 commented Jul 26, 2023

Why don't remove freezing on place so you can change your mind and cancel the ability?

By itself its a worse solution. If on on top of this, its unnecessary.
I like that the reward for dodging out of it is getting to shoot the briefly rooted rav which gets punished because it fucked up - better than the rav just cancelling and immediately following you. Wouldn't feel "right"/as satisfying without the root either but maybe thats just me.

@github-actions
Copy link
Contributor

github-actions bot commented Aug 4, 2023

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

@github-actions github-actions bot added the Stale beg a maintainer to review your PR label Aug 4, 2023
@github-actions github-actions bot closed this Aug 12, 2023
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. Stale beg a maintainer to review your PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants