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

Improves Simple Smile's Work Rates #2380

Merged
merged 2 commits into from
Aug 11, 2024
Merged

Conversation

tile4
Copy link
Contributor

@tile4 tile4 commented Aug 9, 2024

About The Pull Request

Changes Simple Smile's workrates to have an 85% base chance and decrease by 20% instead of 80% and 30% respectively

Why It's Good For The Game

Simple Smile's workrates have been quite outdated throughout the changes we have gotten. With the slower pace of the game, it would be way less plausable for people to have 1 in all. Hopefully this will help make Simple smile better.

Changelog

🆑
balance: rebalanced Gone With a Simple Smile
/:cl:

@Gboster-0 Gboster-0 added the Balance Improvements towards balance of the game. label Aug 9, 2024
@InsightfulParasite InsightfulParasite added the Isolated Code The files added or edited by this PR have a low chance of conflicts but still need to be reviewed. label Aug 9, 2024
@Coxswain-Navigator
Copy link
Contributor

My only issue with this PR is that even the worst case scenario work rates become right about the average for an abnormality of this risk level. I guess its breach is annoying enough though.

@tile4
Copy link
Contributor Author

tile4 commented Aug 10, 2024

My only issue with this PR is that even the worst case scenario work rates become right about the average for an abnormality of this risk level. I guess its breach is annoying enough though.

Actually, I took a look and I guess I misread what Smile's old values are. I will adjust that and have everything properly updated in info to be just .8 scaling instead of .85 because .7 is too awful overall.

Copy link
Contributor

@Kitsunemitsu Kitsunemitsu left a comment

Choose a reason for hiding this comment

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

Approving this minor change, we'll see what happens.

@Kitsunemitsu Kitsunemitsu merged commit 4552743 into vlggms:master Aug 11, 2024
9 checks passed
@tile4 tile4 deleted the patch-3 branch November 5, 2024 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance Improvements towards balance of the game. Isolated Code The files added or edited by this PR have a low chance of conflicts but still need to be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants