Cyborg movement speed rebalance #17577
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
github-actions
bot
added
the
size/M
ThatFiveGuys
added
C-Balance
|
Created discussion thread: https://forum.ss13.co/showthread.php?tid=22156 |
github-actions
bot
added
the
S-Merge-Conflict
…nto borgspeedoverhaul
github-actions
bot
removed
the
S-Merge-Conflict
mordent-goonstation
requested changes
Jan 15, 2024
There was a problem hiding this comment.
https://forum.ss13.co/showthread.php?tid=22156&pid=200165#pid200165
Hold on the no-arms case being considered. Broadly fine with the changes as described, may need some more tweaking and fine-tuning after live play. I do not think this needs test merging, we can adjust iteratively.
|
Changes requested implemented |
|
BigDocOfSpeed.xlsx |
goonstation-issuebot
added
the
S-Testmerged
This was referenced Jan 26, 2024
Closed
This was referenced Feb 10, 2024
Closed
Closed
Closed
goonstation-issuebot
removed
the
S-Testmerged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
About the PR
I'd like to preface this by saying that I think this should be testmerged before merging, I cannot do sufficient testing besides making sure my code actually does what I want it to do and the speed feel right on a local. I think my values are good, but it's hard to say without seeing them in practice.
This pr changes the movement modifier of nearly every single borg part in the game. Below is a chart of all the previous and new values, alongside a few presets (the rp meta is light borg w/ screenhead + treads). The human speed sus. is the speed of a base full health human after 10 tiles (the speed below that is 1.7).
The key playerside changes are as follows, each heavier part now has an increasing effect on movement speed. Light borg's movement speed is unaffected, but light borg with treads is ~30% slower. Every borg above light borg with the exception of heavy borg w/ treads is faster, but treads have relatively less impact on speed. Thrusters are now slightly better speed wise then treads in every scenario.
For code side changes, weight no longer effects movement speed as it just makes balancing these values harder to minimal benefit. The weight variable is left in code in case anyone wants to use them for future borg expansions, however that variable is not used for anything. Any dev tell me if you want it removed and I will do so.
Why's this needed?
In short, light borgs w/ treads are too fast, and heavier borgs are too slow. This should make heavier borgs more playable, and light borgs less desirable. Also its kinda weird that standard chests are objectively better then light chests in every way, like that isnt right.
Changelog