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

AI holograms can switch between holopads more seamlessly #28450

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

Conversation

Vi3trice
Copy link
Contributor

What Does This PR Do

Makes the hologram transfer on holopads for AI more flexible by changing where the area check is done for it specifically, while other mobs are still constrained to the area they're calling to. This prevents cases where the hologram gets cancelled when changing areas even though you're still within range of the old one.

Why It's Good For The Game

The way transfering holograms for AI was too rigid, and didn't make sense for it. Old comment made it sound like it was a perf save, but the distance check was done all the time regardless and looped through all holopads.

Images of changes

TurnOnYourCamera.mp4

Testing

Used holopads as an AI and moved around.


Declaration

  • I confirm that I either do not require pre-approval for this PR, or I have obtained such approval and have included a screenshot to demonstrate this below.

Changelog

🆑
tweak: Range check for holopad projections is more flexible for AI
/:cl:

@Ahzpy Ahzpy mentioned this pull request Feb 19, 2025
1 task
@ParadiseSS13-Bot ParadiseSS13-Bot added the -Status: Awaiting approval This PR is waiting for approval internally label Feb 19, 2025
@Burzah Burzah added the Tweak This PR tweaks something ingame label Feb 20, 2025
@ParadiseSS13-Bot ParadiseSS13-Bot added -Status: Awaiting review This PR is awaiting review from the review team and removed -Status: Awaiting approval This PR is waiting for approval internally labels Feb 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-Status: Awaiting review This PR is awaiting review from the review team Tweak This PR tweaks something ingame
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants