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

[MIRROR] Increased odds of station traits a little. Introduced a "budget", so smaller traits only take half as much space. #1638

Merged
merged 5 commits into from
Feb 28, 2024

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#298
Original PR: tgstation/tgstation#80211

About The Pull Request

Recently, I chatted with others about how few station traits are rolled on a round by round basis - about 59% of the shifts go without either positive or negative traits for example - and how the mild most of these traits are (not a bad thing per se), which results in an underwhelming feature, despite the more interesting bits of it. So, after sharing opinions, I've decided to make this PR to increase the rolls and odds a bit.

EDIT: After reading comments and taking some time to think this thoroughfully, I've decided to push the probabilities back a little in favor of a simple budget system for station traits, to allow for smaller things to only count as half a station trait. This mean smaller traits won't necessarily stop "better" ones from rolling, or at least allow for plentier permutations of traits that do not affect the round TOO much.
I've also reduced the weight of the glitched pda beeps trait from 15 to 10, the same of scarves, wallets and colored assistant jumpsuits.

Why It's Good For The Game

I believe the current odds of station traits to be a smidge low, and that the lack of any sort of cost-budget for station traits to hurt the rarer, more interesting traits (and the feature in general) if the more common, milder ones take just as much space. It's totally within the spirit of the feature to have small, niche traits, though they can get quite boring pretty fast on their own, so what I'm saying is that their cost should stay low so that other traits can roll.

Changelog

🆑 Ghommie
refactor: Introduced a simple budget system to station traits, so that smaller things only count as half a trait, for example.
balance: Increased the odds and maximum number of station traits that can be rolled each shift.
/:cl:

…get", so smaller traits only take half as much space. (#298)

Increased odds of station traits a little. Introduced a "budget", so smaller traits only take half as much space.

Co-authored-by: Ghom <[email protected]>
Co-authored-by: Bloop <[email protected]>
@Steals-The-PRs Steals-The-PRs added Mirroring conflict git cherry-pick во что-то уткнулся. Не ставить вручную, только для бота TG Mirror Balance Refactoring labels Jan 22, 2024
Copy link
Contributor

github-actions bot commented Feb 1, 2024

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 label Feb 1, 2024
@Iajret Iajret removed the Stale label Feb 4, 2024
Copy link
Contributor

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 label Feb 12, 2024
@github-actions github-actions bot closed this Feb 19, 2024
@Iajret Iajret reopened this Feb 27, 2024
@Iajret Iajret merged commit 4a60f12 into master Feb 28, 2024
43 checks passed
@Iajret Iajret deleted the upstream-mirror-298 branch February 28, 2024 18:50
AnywayFarus added a commit that referenced this pull request Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance Config Mirroring conflict git cherry-pick во что-то уткнулся. Не ставить вручную, только для бота Refactoring Stale TG Mirror
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants