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

Converts oil for borgs into a status effect and rebalances it #17618

Merged
merged 7 commits into from Jan 20, 2024

Conversation

ThatFiveGuys
Copy link
Contributor

@ThatFiveGuys ThatFiveGuys commented Jan 11, 2024

About the PR

If you want the tldr of the new oil behavior, in short being oiled recently makes you freshly oiled which makes you go really fast and power efficent for a max of 15 seconds. This decays into the oiled status effect for a max of 6 minutes, which makes you go go somewhat fast and somewhat power efficent. Just look at the video below
https://github.com/goonstation/goonstation/assets/73148980/0258ad07-86d7-4330-8d11-113844f35392

To sum up the old oil behavior for probably unneeded context, one unit of oil applied 2 seconds of being oiled, this status effect reduces your speed by .5, reduces your max stun by 25% + stun resistance by 25, and halves your passive power consumption. This had no clamp.

To sum up the new behavior, this pr splits oil into two status effects, oiled and freshly oiled. Both status effects apply the stun modifiers described above.

Every unit of oil applied to a borg applies half a second of the freshly oiled status effect. This status effect caps out at 15 seconds, meaning you need to apply 30 units to cap this effect. This halves your passive power consumption, and reduces your movement delay by 50%. When freshly oiled is finished, you get applied the oiled status effect with 24 times the duration you've had the freshly oiled status effect for, meaning you have to have freshly oiled for at least 15 seconds to cap out at 6 minutes.

Oiled status effect just reduces your movement delay and passive power consumption by 15%, and also does the same stun modifiers mentioned above.

Why's this needed?

This new behavior is both far more readable, and also far more interesting and fun, also hopefully well balanced.

Changelog

(u)Ikea
(*)The previous behavior for oil in regards to borgs has been split into two status effects, freshly oiled and oiled. These apply speed increase and power consumption decrease in varying amounts, alongside stun resistance.

@boring-cyborg boring-cyborg bot added the C-Sprites Automatically applied on any .dmi or icons folder change label Jan 11, 2024
@github-actions github-actions bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jan 11, 2024
@ThatFiveGuys ThatFiveGuys added C-Balance Balance changes, buffs and nerfs E-Input-Wanted Input and feedback are wanted. Also posts a discussion thread on the forums. C-Rework Reworks a feature E-Add-To-Wiki A PR that will require changes to the wiki A-Silicons Deals with the angry metal robots labels Jan 11, 2024
Copy link
Contributor

Created discussion thread: https://forum.ss13.co/showthread.php?tid=22177

@ZeWaka
Copy link
Member

ZeWaka commented Jan 11, 2024

gib bideo u posted in imcoder

@ThatFiveGuys
Copy link
Contributor Author

gib bideo u posted in imcoder

It was in imspriter you fool, also added

@ThatFiveGuys ThatFiveGuys changed the title Converts oil for borgs into a status effect Converts oil for borgs into a status effect and rebalances it Jan 11, 2024
@TDHooligan
Copy link
Contributor

notable it nerfs light borgs' speed bonus more than it nerfs heavy borgs. i like this more than the existing effect

ThatFiveGuys and others added 4 commits January 14, 2024 18:33
Co-authored-by: ZeWaka <zewakagamer@gmail.com>
Co-authored-by: ZeWaka <zewakagamer@gmail.com>
Co-authored-by: ZeWaka <zewakagamer@gmail.com>
@ZeWaka ZeWaka added the S-Ready-For-Final-Review This PR has been looked over by members of the community. (automatic after 2 approvals) label Jan 15, 2024
Copy link
Member

@Tarmunora Tarmunora left a comment

Choose a reason for hiding this comment

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

at 15 seconds, the "freshly oiled" thing seems pretty pointless

code/mob/living/silicon/robot.dm Outdated Show resolved Hide resolved
@ThatFiveGuys
Copy link
Contributor Author

at 15 seconds, the "freshly oiled" thing seems pretty pointless

Freshly oiled is mostly intended to be a short speed bonus for borgs to apply on themselves sorta similar to how current behavior of oil can be utilized. Also borgs going mach 12 funny.

@ZeWaka
Copy link
Member

ZeWaka commented Jan 17, 2024

at 15 seconds, the "freshly oiled" thing seems pretty pointless

Freshly oiled is mostly intended to be a short speed bonus for borgs to apply on themselves sorta similar to how current behavior of oil can be utilized. Also borgs going mach 12 funny.

Yeah, I also thought of/requested/proposed this because it adds a bit of complexity to an otherwise boring "set and forget" feature

@ThatFiveGuys
Copy link
Contributor Author

Anyways fixed the inconsistency, I have no clue why that part of borg code has so much repeated code anyways, but oh well. Anyways, for some additional context, I did some tests on oshans with light borgs and with the level of speed freshly oiled offers you, you can do half a loop around oshan as a light borg in 15 seconds. Anyways if its a decided no then just tell me and Ill axe freshly oiled, but I think its interesting and neat.

@frawhst frawhst merged commit 4a10dab into goonstation:master Jan 20, 2024
24 checks passed
github-actions bot pushed a commit that referenced this pull request Jan 20, 2024
@Studenterhue Studenterhue removed the E-Add-To-Wiki A PR that will require changes to the wiki label Jan 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Silicons Deals with the angry metal robots C-Balance Balance changes, buffs and nerfs C-Rework Reworks a feature C-Sprites Automatically applied on any .dmi or icons folder change E-Input-Wanted Input and feedback are wanted. Also posts a discussion thread on the forums. S-Ready-For-Final-Review This PR has been looked over by members of the community. (automatic after 2 approvals) size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants