Skip to content
This repository was archived by the owner on May 22, 2025. It is now read-only.

Reduce dynamic weight to 0 - #22136

Merged
ToasterBiome merged 1 commit into
yogstation13:masterfrom
Moltijoe:ExpungeDynamicFromCodebase
Jun 2, 2024
Merged

ToasterBiome merged 1 commit into
yogstation13:masterfrom
Moltijoe:ExpungeDynamicFromCodebase

Conversation

@Moltijoe

@Moltijoe Moltijoe commented May 31, 2024 •

Copy link
Copy Markdown
Contributor

Why is this good for the game?

Dynamic has not only basic issues
-overly complicated
-bug prone
But it doesn't even get close to solving the problem it attempts to solve because we don't commit to it 100%
having both it and gamemodes results in us flip flopping between traitor and dynamic,
this ends up with us just always having traitor as dynamic is almost always going to be traitor at this pop
in a sense, it makes the problem worse

anyways, it's not in a state that's worth committing 100% to
Spending dev time to make it good makes little sense when it'd be better for the longrun to port monke storytellers
I do not have permission to port monke storytellers for the time being

i considered removing it from the code outright, but until we have an alternative, i'm sure biome would want me to keep it for the downstreams

so disabling dynamic is the best course of action in my mind

Testing

number change innit

🆑
rscdel: Disables dynamic
/:cl:

@Moltijoe
Moltijoe requested a review from a team as a code owner May 31, 2024 08:44
@github-actions github-actions Bot added the Config Config files need to be changed on the host for this to work label May 31, 2024
@ToasterBiome
ToasterBiome merged commit a4a34ea into yogstation13:master Jun 2, 2024
@Yogbot-13 Yogbot-13 added the Revert / Deletion Altoids is probably mad about this label Jun 2, 2024
Yogbot-13 added a commit that referenced this pull request Jun 2, 2024
@Moltijoe
Moltijoe deleted the ExpungeDynamicFromCodebase branch June 2, 2024 04:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Config Config files need to be changed on the host for this to work Revert / Deletion Altoids is probably mad about this

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants