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

Storyteller rebalance and modularization. (And two more storytellers) #549

Merged
merged 13 commits into from
Nov 24, 2023

Conversation

KittyNoodle
Copy link
Contributor

@KittyNoodle KittyNoodle commented Nov 13, 2023

About The Pull Request

Modularizes all the event overrides.

Adds some extra tags to some antags(read: Ninja and carp)

Jester+Warrior are now medium/highpop only.

Adds two storytellers:

The Hermit

The Hermit will create mostly internal conflict around the station, and rarely any external threats.

The Nobleman

The Nobleman enjoys a good fight but abhors senseless destruction.

Why It's Good For The Game

Jester + Warrior just spam the same events over and over again on lowpop, causing people to get burnt out easily.
The reason for the player limit of 40 is because its based on server pop, not readied players, the amount of players readied is usually ~10 less than server pop.

Ninja and carp are both destructive events.

The Nobleman creates a good balance between the warrior's combat focus and the sleeper's focus on non-destructive gameplay that allows non-combatants to enjoy the game too.

The Hermit is good for those who prefer intrigue and internal conflict.

Changelog

🆑
add: The Observer, a more roundstart focused storyteller that prefers to watch idly
add: The Nobleman, a less destructive variant of warrior that lets pacifists enjoy the game too.
qol: No more lowpop kudzu spam
balance: Jester and warrior are now highpop/midpop only.
balance: Changed a lot of storyteller tags to make more sense
code: Modularizes a lot of storyteller code
/:cl:

@KittyNoodle KittyNoodle marked this pull request as ready for review November 13, 2023 21:56
/datum/round_event_control/operative
track = EVENT_TRACK_MAJOR
tags = list(TAG_DESTRUCTIVE, TAG_COMBAT, TAG_EXTERNAL)
checks_antag_cap = TRUE
Copy link
Collaborator

Choose a reason for hiding this comment

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

This should not check antag cap as its meant to be a safeguard.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Also might be good to remove its tags.

@dwasint dwasint merged commit 553078e into Monkestation:master Nov 24, 2023
github-actions bot added a commit that referenced this pull request Nov 24, 2023
@KittyNoodle KittyNoodle deleted the storytellersv2 branch December 20, 2023 23:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants