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] Fixes Hatsune Migo Spawning Comments and Code #2864

Merged
merged 1 commit into from
Mar 12, 2024

Conversation

NaakaKo
Copy link
Collaborator

@NaakaKo NaakaKo commented Mar 10, 2024

Original PR: https://api.github.com/repos/tgstation/tgstation/pulls/81856

About The Pull Request

The prob() was not lining up with the comment, prob(0.04) is NOT 1/20 odds. it's also wack to have 1/20 odds for something meant to be super duper cool and rare (lest people get sick of the joke), so I balanced it to be a prob(0.1) odds as well (1/1000 i think).

i also don't like how non-special the mi-go replacement was and detested the fact that the desc would duplicate if an admin spawned in a mi-go twice (mentioning the blue hair twice), so let's make it a tad bit more sane by actually replacing it with the type introduced in that PR and checking against that type.

Why It's Good For The Game

image

people would likely try to fix the way this worked in the future to match the comment and that's icky imo. let's rebalance it to be more sane and also clean up the code

Changelog

the odds of players having seen this sprite in game were already really rather low so let's not bother them with the tantalizing unknown

@NaakaKo NaakaKo added the Upstream PR Merged Used by the mirror bot. label Mar 10, 2024
@CliffracerX CliffracerX merged commit b45866b into master Mar 12, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Upstream PR Merged Used by the mirror bot.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants