Skip to content

PAI's can pilot Modsuits now#534

Open
Eponymic-sys wants to merge 4 commits intoProjectOmu:masterfrom
Eponymic-sys:PAI_Modsuit_Pilot
Open

PAI's can pilot Modsuits now#534
Eponymic-sys wants to merge 4 commits intoProjectOmu:masterfrom
Eponymic-sys:PAI_Modsuit_Pilot

Conversation

@Eponymic-sys
Copy link

About the PR

Modsuit helmets get a 1x2 storage slot with a PAI whitelist, active PAIs can pilot when placed inside

Why / Balance

Kinda funny?

Technical details

Using existing systems, nothing crazy

Media

pai.mp4

Small fixes/refactors are exempt. Media may be used in SS14 progress reports with credit. -->

Requirements

Breaking changes

N/A

Changelog

@github-actions github-actions bot added S: Untriaged Has not been set a status; currently not labeled. Changes: No C# Purely YAML/non-C#. and removed S: Untriaged Has not been set a status; currently not labeled. size/S labels Jan 20, 2026
Copy link
Member

@CliveOcelot CliveOcelot left a comment

Choose a reason for hiding this comment

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

Tested, works:
To note:
PAI cannot pilot under these conditions:
Owner is asleep (ex: under effect of nocturine)
Owner is stunned
Owner is crit/dead

It does seem to disable movement prediction for both people while the pai is in the helmet, but that isn't a major issue.

@CliveOcelot CliveOcelot added the S: Needs Second Approval Approved by another maintainer, just needs one more to check it. label Jan 21, 2026
@Eponymic-sys
Copy link
Author

Do you think the would pass muster upstream? I don't mind making a PR there if it's easier.

@Cygnye Cygnye added S: Approved This PR has been approved. and removed S: Needs Second Approval Approved by another maintainer, just needs one more to check it. labels Feb 23, 2026
Copy link
Member

@CliveOcelot CliveOcelot left a comment

Choose a reason for hiding this comment

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

After further discusion, this needs to be made into a proper predicted system, and movement should be locked to intelicard.

@CliveOcelot CliveOcelot added S: Needs Changes Changes/Concerns have been brought up and should be made before review. and removed S: Approved This PR has been approved. labels Feb 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changes: No C# Purely YAML/non-C#. S: Needs Changes Changes/Concerns have been brought up and should be made before review. size/S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants