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

Conversation

@Xoxeyos
Copy link
Contributor

@Xoxeyos Xoxeyos commented Mar 15, 2021

General Documentation

Intent of your Pull Request

This ports over Dynamic 2021, and Ghost Pool Protection, which is sort of wrapped up with it, it splits the dynamic budget into two separate budgets, one for roundstart antagonists, and latejoin / midround antagonists, and a bunch of other changes. Here's the two PR's that are being ported.
BeeStation/BeeStation-Hornet#3797 - Dynamic 2021
BeeStation/BeeStation-Hornet#3803 - Ghost Pool Protection

Why is this change good for the game?

Dynamic uses less threat during the start of the round, and adds whatever is left over into the midround budget, certain events can hijack other random events replacing one event with another, say a sentience event with an alien infestation, of course relying on the budget. Highpop requirements have been removed, since they were sort of useless.

https://user-images.githubusercontent.com/35135081/109071468-9cab7e00-76a8-11eb-8f9f-2b920c602ef4.png
https://cdn.discordapp.com/attachments/712496193183154207/820869133494779904/unknown.png

Wiki Documentation

Briefly describe your PR and the impacts of it, in layman's terms.

This will be the basis of the Wiki entry for your PR, and more information / detail is better for Wiki editors to integrate.

Dynamic will now split the threat into two separate budgets, one for roundstart, one for latejoiners and midrounders, more logging has been added for dynamic; The ghost pool protection allows admins to restrict what players can choose for ghostroles, say in case of an event, or to prevent griefing in some sort of way. The readme.md in game/gamemodes/dynamic/ is now alot more detailed.

What should players be aware of when it comes to the changes your PR is implementing?

Dynamic is now split into two separate budgets, NT announcements regarding the threat level can be wrong, certain events can hijack other events.

What general grouping does this PR fall under?

Gamemode changes, dynamic update, admin logging and tools.

Are there any aspects of the PR that you would like us not to mention on the Wiki?

Leaving the hijacking out and the threat level being faked might be nice, the ghost pool protection wouldn't important for anyone but admins.

If there are any numerical values involved in your PR that will be relevant to a player, please note them here.

I don't believe there are any.

Changelog

🆑 Xoxeyos, TheChosenEvilOne, Rohesie, IndieanaJones, Mothblocks, Qustinnus, tralezab, Theos
del: Removed high population override from dynamic.
fix: Dynamic can now be configured when made the forced secret mode.
tweak: dynamic threat reports are no longer perfectly accurate, they diverge and are sometimes downright wrong.
add: Dynamic now rolls a separate budget for round start and midround antagonists, meaning less round start antags will roll in favor of more midround ones.
admin: The antag_cap value now uses an equation that scales with population, rather than a fixed array.
admin: autotraitor_cooldown now uses deciseconds rather than ticks, so the older value of 450 now means 45 seconds. Set it to be 9000 if you want it to be accurate to the old value (15 minutes), or simply unset it.
admin: Midround/latejoin dynamic timers are now configurable.
admin: Syndicate Sleeper Agent will now give details when it fails.
fix: The autotraitor cooldown now respects configuration, rather than resetting back to 15 minutes after it is rolled.
fix: Fixed dynamic parameters not checking the configuration. This means that Manuel will now have slower midrounds, as was intended.
fix: Fixed multiple high impact rulesets being chosen on Dynamic.
fix: Midround dynamic injection no longer has a 100% chance of activating.
add: Dynamic will now play a part in controlling random event antagonists. If a dynamic midround injection is coming too soon, or too early, then the next midround injection will be buffed. Otherwise, it'll spawn normally.
admin: Admins can now cancel or replace midround rulesets.
fix: Admins can now correctly force midround rulesets.
admin: New Adminbus button- Ghost Pool Protection! It lets you enable and disable which sources ghosts can rejoin the round, minus their physical bodies getting revived.
/:cl:

Xoxeyos added 2 commits March 14, 2021 03:05
Good luck to myself, Xoxeyos
Admin.dm, rust_g, game.dm, mobs.dm, pai.dm, dynamic folder, holy_weapons.dm, admin_verbs.dm, topic.dm, ghost_pool_protection, antag_datum.dm, CTF.dm, corpse.dm, _event.dm, living/brain/posibrain.dm, giant_spider.dm, dynamic.json
@Xoxeyos Xoxeyos requested a review from a team as a code owner March 15, 2021 04:47
@Xoxeyos Xoxeyos requested a review from a team March 15, 2021 04:47
@github-actions github-actions bot added the Admin This PR affects administrators label Mar 15, 2021
@Xoxeyos
Copy link
Contributor Author

Xoxeyos commented Mar 16, 2021

A test merge would be needed for this, incase there wasn't something caught while testing it.

@SomeguyManperson SomeguyManperson added TESTMERGED This PR is currently live in a test merge. Test Merge - Requested This PR is waiting for a test merge to be scheduled and removed Test Merge - Requested This PR is waiting for a test merge to be scheduled TESTMERGED This PR is currently live in a test merge. labels Mar 16, 2021
Copy link
Contributor

@redmoogle redmoogle left a comment

Choose a reason for hiding this comment

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

urgent fixes. bare minimum append these changes

@redmoogle redmoogle dismissed their stale review March 18, 2021 23:19

Fixed bugs myself

@Xoxeyos
Copy link
Contributor Author

Xoxeyos commented Mar 20, 2021

Thanks for fixing it, sorry I didn't get around to it.

@JamieD1 JamieD1 added the Test Merge - Scheduled This PR is waiting to be test merged label Mar 20, 2021
@Xoxeyos
Copy link
Contributor Author

Xoxeyos commented Mar 21, 2021

I'm going to be removing the 'adminbus' tab and I'll be putting the ghost pool protection into the admin tab.

@Yogbot-13
Copy link
Contributor

Test Merge Updated

Server Instance

main

Merged By

JamieD12

Revision

Origin: 92e694a
Pull Request: 603b047
Server: da56d524b85eeabf513b808c3ed6a8c9df605f9a

@Yogbot-13
Copy link
Contributor

Test Merge Updated

Server Instance

main

Merged By

JamieD12

Revision

Origin: 6eac6b8
Pull Request: 603b047
Server: 92b18435786db50501d15b57a7d0f399158dcce7

@Yogbot-13
Copy link
Contributor

Test Merge Updated

Server Instance

main

Merged By

JamieD12

Revision

Origin: 6eac6b8
Pull Request: 603b047
Server: 4cf045dcc80de3751ed501a3fdf71a161b4b6078

@Yogbot-13
Copy link
Contributor

Test Merge Updated

Server Instance

main

Merged By

JamieD12

Revision

Origin: 09dc773
Pull Request: 603b047
Server: 8ce5531e4aa90f730f2881ca15125742069cc3b7

@Yogbot-13
Copy link
Contributor

Test Merge Updated

Server Instance

main

Merged By

JamieD12

Revision

Origin: 50e22d9
Pull Request: 603b047
Server: cfbef971e39f756f57c6351ccba16176fb7ca9b7

@Yogbot-13
Copy link
Contributor

Test Merge Updated

Server Instance

main

Merged By

JamieD12

Revision

Origin: d244ed8
Pull Request: 603b047
Server: c9dd238bda1b13104771876231c7f061360f117a

@Yogbot-13
Copy link
Contributor

Test Merge Updated

Server Instance

main

Merged By

JamieD12

Revision

Origin: b34fc83
Pull Request: 603b047
Server: 99a8136a2c3ea2386b7417cadce313eef7205037

@Yogbot-13
Copy link
Contributor

Test Merge Updated

Server Instance

main

Merged By

JamieD12

Revision

Origin: 51d15f3
Pull Request: 603b047
Server: 08a5b01226a862a638855658a1fcb349f2180bdf

@adamsong adamsong added the TESTMERGED This PR is currently live in a test merge. label Jul 2, 2021
@Yogbot-13
Copy link
Contributor

Test Merge Updated

Server Instance

main

Merged By

JamieD12

Revision

Origin: faf404b
Pull Request: 603b047
Server: 261ac2b415670d9615de140a821f0aed25f45412

@Yogbot-13
Copy link
Contributor

Test Merge Updated

Server Instance

main

Merged By

JamieD12

Revision

Origin: dfdf9d2
Pull Request: 603b047
Server: a57b89a5cefc62a3059460567fc50d2b23efa5f5

@Yogbot-13
Copy link
Contributor

Test Merge Updated

Server Instance

main

Merged By

JamieD12

Revision

Origin: dfdf9d2
Pull Request: 603b047
Server: d723368c137b9977e75b3e26dfb321c788297cdd

@Yogbot-13
Copy link
Contributor

Test Merge Updated

Server Instance

main

Merged By

JamieD12

Revision

Origin: 24d1f23
Pull Request: 603b047
Server: 1c84e2f8d3d922d24bcc92d7edb9ce9e32d5252e

@Yogbot-13
Copy link
Contributor

Test Merge Updated

Server Instance

main

Merged By

JamieD12

Revision

Origin: e581b72
Pull Request: 603b047
Server: fc7122cf9f35688fd96d1b459cc1c9b61682e752

@Yogbot-13
Copy link
Contributor

Test Merge Updated

Server Instance

main

Merged By

JamieD12

Revision

Origin: 1befeb3
Pull Request: 603b047
Server: 87f7ce98fbe6e2119e87704ba716dc7213e89bd9

@Yogbot-13
Copy link
Contributor

Test Merge Updated

Server Instance

main

Merged By

JamieD12

Revision

Origin: bfcf8bf
Pull Request: 603b047
Server: e56275b82d8a1b0a3a432a233629ac206864f83a

@Yogbot-13
Copy link
Contributor

Test Merge Updated

Server Instance

main

Merged By

JamieD12

Revision

Origin: 074dc71
Pull Request: 603b047
Server: e10404c9a10c2ee7f917b59885fa18ede7f0435a

@Yogbot-13
Copy link
Contributor

Test Merge Updated

Server Instance

main

Merged By

JamieD12

Revision

Origin: aeb34a7
Pull Request: 603b047
Server: 3fc96da3fe7b326106f45bec9fcd29d2afa4b3c4

@Yogbot-13
Copy link
Contributor

Test Merge Updated

Server Instance

main

Merged By

JamieD12

Revision

Origin: 5b96721
Pull Request: 603b047
Server: 0e5c469765b215fb6c9c7efe7e82d2ab360f0212

@Yogbot-13
Copy link
Contributor

Test Merge Updated

Server Instance

main

Merged By

JamieD12

Revision

Origin: a2b0d80
Pull Request: 603b047
Server: 48ff5736e8e42a68fceb7e361203ebf0fa1bbeb2

@Yogbot-13
Copy link
Contributor

Test Merge Updated

Server Instance

main

Merged By

JamieD12

Revision

Origin: 75e2418
Pull Request: 603b047
Server: a573d4fda4232a30130441438b8607ed178df93e

@Yogbot-13
Copy link
Contributor

Test Merge Updated

Server Instance

main

Merged By

JamieD12

Revision

Origin: b0848ca
Pull Request: 603b047
Server: 46c827dc9eeae1ddda6382961b987de97833563a

@JamieD1 JamieD1 merged commit 4374b93 into yogstation13:master Jul 9, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Admin This PR affects administrators Test Merge - Requested This PR is waiting for a test merge to be scheduled Test Merge - Scheduled This PR is waiting to be test merged TESTMERGED This PR is currently live in a test merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants