Skip to content

Human AI now Compare Vehicle Factions to their Neutral and Friendly Factions. #989

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

Merged

Conversation

The32bitguy
Copy link
Contributor

@The32bitguy The32bitguy commented May 27, 2025

About the pull request

Human Ai now check if the faction of a vehicle is the same, friendly or neutral to them.
Before they would only check if the faction was the same, which made them always shoot vehicles of otherwise friendly or neutral players as defined by the GM in the faction manager panel.
As a reminder, vehicles faction and IFF are based on the faction of the last human to have used the driver seat.

Remember: something that is self-evident to you might not be to others. Explain your rationale fully, even if you feel it goes without saying. -->

Explain why it's good for the game

Fixes an oversight. Makes it not needed to var edit a vehicle to stop Human Ai shooting at what you may have already designated as a friendly faction

Changelog

🆑
fix: Human AI now check if a vehicle is in their friendly or neutral factions before targeting.
/:cl:

@cm13-github
Copy link
Contributor

cm13-github commented May 28, 2025

Test merge deployment history:

Test Merge Deployed @ 05/28/2025 00:57:15 +00:00:
Server Instance

PVECM: Red

Merged By

Rob606

Revision

Origin: 4f91864
Pull Request: 32f6d21
Server: ec51c2f305adde700fb5e610cc43a7df6cdb8ca0

Test Merge Deployed @ 05/28/2025 18:12:51 +00:00:
Server Instance

PVECM: Green

Merged By

EmiSol

Revision

Origin: 0fe7029
Pull Request: 32f6d21
Server: 2472877b911782574dc81fd48abce8b86f6a2e50

Test Merge Updated @ 05/28/2025 21:33:29 +00:00:
Server Instance

PVECM: Red

Merged By

Rob606

Revision

Origin: 0fe7029
Pull Request: 32f6d21
Server: cd3a4afad87fb40ef90e57c8c3247d75ad11b03c

Test Merge Updated @ 05/29/2025 01:03:44 +00:00:
Server Instance

PVECM: Red

Merged By

Karmac

Revision

Origin: 0fe7029
Pull Request: 32f6d21
Server: 1440032fc76c6848657121c6c3149432e1705964

Test Merge Updated @ 05/29/2025 05:38:10 +00:00:
Server Instance

PVECM: Red

Merged By

Some Random Username

Revision

Origin: 0fe7029
Pull Request: 32f6d21
Server: cf57a76dde2fc68779cff97d753bc417e3b07e3f

Test Merge Updated @ 05/29/2025 06:36:30 +00:00:
Server Instance

PVECM: Green

Merged By

Eonoc

Revision

Origin: 0fe7029
Pull Request: 32f6d21
Server: f018f078c9c7303386d20b7038c994708b7c5fdf

Test Merge Updated @ 05/29/2025 13:21:06 +00:00:
Server Instance

PVECM: Green

Merged By

Suno

Revision

Origin: 0fe7029
Pull Request: 32f6d21
Server: 783a1f6e0f6a1cc0fdd21b14c16552a6cf3e85db

@Zonespace27 Zonespace27 enabled auto-merge (squash) May 29, 2025 13:48
@Zonespace27 Zonespace27 merged commit 2973020 into cmss13-devs:master May 29, 2025
42 checks passed
cmss13-ci bot added a commit that referenced this pull request May 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants