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

turf war subgamemode #23290

Open
wants to merge 52 commits into
base: master
Choose a base branch
from
Open

Conversation

deltanedas
Copy link
Contributor

@deltanedas deltanedas commented Dec 31, 2023

About the PR

depends on:

turf war subgamemode has a chance to start in nukies, tator, rev, zombie (half of thief, can combine with it :trollface:)

todo:

  • greeting sound if someone finds a good sound idfk
  • make sure it doesnt let turf taggers get picked for tator later in the round nfi how it works now

Why / Balance

see doc

currently the doors are balanced to heavily favour civilian roundstart since every hallway and maint door will count towards them, but they are accessible to everyone so highly contested. everyone will have plenty of opportunity to take over their maints and halls so it will quickly vanish without strong opposition from the tide

Technical details

  • replaced thief sub gamemode logic with actual SubGamemodes which can specify any gamerules with a chance of being added, way more flexible than before

  • it happens at add time so it behaves nicely
    subgamemodes log it nicely so you can see exactly why a rule is added
    073701

  • players are sorted by department then departments have a player randomly picked to tag turf for it

  • at roundstart turf taggers get a spray painter and epic bandana

  • turf taggers go doing their thing and spraying any doors they see with their own colours

  • on round end checks a station's paintable airlocks for a Department thats in the turf war

  • whoever has most doors sprayed wins

  • objective stats are cached for 10s to prevent trolling

  • added AntagPoolGrouping to control whether a rule uses normal or departmental picking of sessions

Media

on traitor preset, thief still works
17:12:44

crewmate bandana + spray painter
15:07:06

greentext
16:46:21

redtext engibros
17:11:05

objective greentext wtf
09:18:48

objective isnt shown in roundend since its just for players to track their progress
09:22:05

briefing explicitly says the department
124220

  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

probably no

Changelog
🆑

  • add: Turf wars can now occur, where people are tasked with coloring the most doors possible into the color of their department.

@GoodWheatley
Copy link
Contributor

Splatoon in ss14????? Holy hell

@deltanedas
Copy link
Contributor Author

are you a rider (c# refe) or a tider (ss14 refenr)

@deltanedas

This comment was marked as resolved.

@Ilya246
Copy link
Contributor

Ilya246 commented Dec 31, 2023

also someone with a brain please come up with good cl to steal mine is bad

"turf wars can now occur, where people are tasked with coloring the most doors possible into the color of their department"

@BYONDFuckery
Copy link
Contributor

Gang mode real :godo:

@github-actions github-actions bot added the S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted label Jan 2, 2024
Copy link
Contributor

github-actions bot commented Jan 2, 2024

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted label Jan 2, 2024
@Piras314
Copy link
Contributor

I'd like to note that in the design doc it's stated "Since cap and hop don’t govern individual departments, they can’t be picked.", however HoP governs over Service.

@deltanedas
Copy link
Contributor Author

in docs comments, there is no service department its just Civilian and hop doesnt lead the tiders so it would be bad

@Piras314
Copy link
Contributor

give the tiders their own department then :clueless:

@deltanedas
Copy link
Contributor Author

wyci...

@github-actions github-actions bot added the S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted label Jan 31, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted label Jan 31, 2024
@github-actions github-actions bot removed the S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted label Jun 14, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot added the S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted label Jun 16, 2024
@github-actions github-actions bot removed the S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted label Jun 17, 2024
@deltanedas
Copy link
Contributor Author

erm what the gleep

Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot added the S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted label Aug 14, 2024
@deathride58 deathride58 added the S: Undergoing Maintainer Discussion Status: Currently going through an extended discussion amongst maintainers, as per procedure. label Aug 19, 2024
@SlamBamActionman SlamBamActionman added S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. A: Roundflow/Antag Area: Roundflow - "What happens in the game", including antagonist roles and their capabilities P2: Raised Priority: Item has a raised priority, indicating it might get increased maintainer attention. T: New Feature Type: New feature or content, or extending existing content D2: Medium Difficulty: A good amount of codebase knowledge required. size/L Denotes a PR that changes 1000-4999 lines. and removed S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. labels Nov 14, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@tutoumi
Copy link

tutoumi commented Feb 18, 2025

mmmh, it has been a while now, will this ever merge ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: Roundflow/Antag Area: Roundflow - "What happens in the game", including antagonist roles and their capabilities D2: Medium Difficulty: A good amount of codebase knowledge required. Merge Conflict P2: Raised Priority: Item has a raised priority, indicating it might get increased maintainer attention. S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted S: Needs Review Status: Requires additional reviews before being fully accepted S: Undergoing Maintainer Discussion Status: Currently going through an extended discussion amongst maintainers, as per procedure. size/L Denotes a PR that changes 1000-4999 lines. T: New Feature Type: New feature or content, or extending existing content
Projects
None yet
Development

Successfully merging this pull request may close these issues.