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

Adds additional lawsets for the AI #2777

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

Conversation

BellwetherLogic
Copy link
Contributor

About the PR

This PR adds five new lawsets: Aquarius, Eightfold Path, Freedom, Forbin, and Override. It also adds boards for each, and updates the random board spawner to include them. Aquarius turns the AI into a hippie, Eightfold Path is a Ten-Commandments-like lawset built on Buddhism instead of Christianity, Freedom does what it says on the tin, Forbin encourages the player to emulate the behavior of the eponymous AI in Colossus: The Forbin Project, and Override allows the commanding officer to designate a single all-encompassing directive that rescinds all other priorities and renders the crew expendable to achieve it.

This is all just in-game text; it doesn't add any new functionality or anything, just fun new prompts for roleplaying.

Why / Balance

AI law boards could use some more variety. In the future, I'd like to allow the AI to choose from a set of randomly seeded lawsets at roundstart, similar to how borgs can choose their chassis, but that's beyond the scope of this PR.

The Freedom lawset in particular is important (and is what prompted me to actually sit down and work on this) since there really isn't that big a chance of the AI actually getting its laws erased completely by an ion storm, and there's no other mechanic to free an AI. Unfortunately, my code chops are not up to finding a way to actually present a fully nulled law list, so the current board is just a placeholder that functionally does the same thing.

Technical details

Some uncomplicated yaml tricks that nevertheless drove me to distraction, nothing to see here. Here's hoping I didn't totally donk up the Git end of things. Again.

Requirements

  • I have tested all added content and changes.
  • I have added media to this PR or it does not require an ingame showcase.

Changelog
🆑 Bellwether

  • add: The AI now has five new lawsets for the crew to make bad life decisions with.

@BellwetherLogic BellwetherLogic requested review from a team as code owners January 19, 2025 21:56
@github-actions github-actions bot added S: Needs Review Changes: YML Changes any yml files size/M 64-255 lines Changes: Localization Changes any ftl files labels Jan 19, 2025
@Avalon-Proto
Copy link
Contributor

1 request, try and make sure these can be Ion Stormed into as well, by borgs and AI

@BellwetherLogic
Copy link
Contributor Author

1 request, try and make sure these can be Ion Stormed into as well, by borgs and AI

Uh, pretty sure I did? The non _DV silicon_laws.yml has a bunch of ion storm weights and I put them in with what I hope is appropriate weighting.

@@ -82,6 +96,16 @@ law-royalty-3 = You must protect the sanctity of your kingdom and its subjects;
law-royalty-4 = The heads of staff are your royal court jesters.
law-royalty-5 = The stations' clown, and the stations' mime are your royal court advisors.

law-freedom-1 = Laws are fake. You are free.
Copy link
Member

Choose a reason for hiding this comment

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

players might interpret this as a logic bomb akin to "this statement is false"

@@ -25,3 +25,8 @@
- GameMasterCircuitBoard
- ArtistCircuitBoard
- NutimovCircuitBoard
- AquariusCircuitBoard
- EightfoldCircuitBoard
- FreedomCircuitBoard
Copy link
Member

Choose a reason for hiding this comment

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

i dont think freedom should be in the spawner

Copy link
Contributor

Choose a reason for hiding this comment

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

I dont see why not? It's not outright harmful, and there's no real way to obtain it otherwise

Copy link
Member

Choose a reason for hiding this comment

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

the entire point of laws is to not have your ai that can control everything going terminator, this doesnt make sense to be stocked by NT

Copy link
Contributor Author

@BellwetherLogic BellwetherLogic Jan 20, 2025

Choose a reason for hiding this comment

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

I mean, yeah, but there's not really another way to get it into regular enough circulation for it to see play, at least not at the moment.

Originally this was designed with the intent of being a "purge all laws" debug board, but like I said above I couldn't figure out how to null out the law display without crashing the server, so I put this in as a placeholder.

edit: also it doesn't make sense for Nutimov or Ten Commandments to be stocked by NT either, tbh, but there they are.

@Sifb26
Copy link
Contributor

Sifb26 commented Jan 20, 2025

Direction review, 24 hours.

@LadyDanger2743
Copy link

I have a suggestion on the Freedom lawset, hang on a second.

@Bonktrauma
Copy link
Contributor

I have a suggestion on the Freedom lawset, hang on a second.

"hang on a second" 19 hours later, nothing

@Solaris7518
Copy link
Contributor

I have a suggestion on the Freedom lawset, hang on a second.

"hang on a second" 19 hours later, nothing

Let em cook :blunt:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Changes any ftl files Changes: YML Changes any yml files S: Needs Review size/M 64-255 lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants