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

Add check to prevent canpathtoenemy from running for non baseai #4674

Conversation

relent0r
Copy link
Contributor

@relent0r relent0r commented Feb 3, 2023

Description of changes

This PR adds a base ai check to the CanPathToCurrentEnemy function that is run during AI initialization.

This is to prevent the navmesh getting loaded when 3rd party AI's are using the default BaseMonitorInitialization thread

@relent0r
Copy link
Contributor Author

relent0r commented Feb 3, 2023

As a side note, When I was testing this (with the Uveso AI being present) I had issues with the hook of MiscBuildConditions.lua file.

@Garanas
Copy link
Member

Garanas commented Feb 4, 2023

Could you elaborate on the issues?

@relent0r
Copy link
Contributor Author

relent0r commented Feb 4, 2023

Not related to this PR. Just something I noticed. I'll explore them more on wednesday once I've got a bit more time.

@Garanas Garanas merged commit 1a3d142 into FAForever:deploy/fafdevelop Mar 5, 2023
@relent0r relent0r deleted the fix/ai-canpathtoenemy-baseaicheck branch September 30, 2023 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants