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

Improvements to Engagement #317

Merged

Conversation

nk3nny
Copy link
Owner

@nk3nny nk3nny commented Apr 18, 2022

Yeah. There is a typo in the branch name.

Changes
Adds groupMemory check when CQB engaging
Adds visibilty check to determine rooms secured
Adds fix to attempting to suppress targets above ground
Improves FSM by making dodging trigger a "can-fire" rather than assessment state.
Fixes use of lookAt to doWatch

Description
Generally improves use of lookAt which fixes AI getting information they are not supposed to have.

Better, more consistent handling og groupMemory, meaning that AI will more effectively clear buildings as teams. Also tweaks many settings to ensure infantry move more intelligently to check buildings. Including visual checks to determine if building positions are cleared. This fixes units 'clearing' buildings from the outside.

Adds groupMemory check when CQB engaging
Adds visibilty check to determine rooms secured

Adds fix to attempting to suppress targets above ground
Improves FSM by making dodging respond to fire-states (not assess)
Improves FSM with more closely requiring cqb checks
Fixes use of lookAt to doWatch

Generally improves use of lookAt which fixes AI getting information they are not supposed to have.

Better, more consistent handling og groupMemory, meaning that AI will more effectively clear buildings as teams. Also tweaks many settings to ensure infantry move more intelligently to check buildings.
Wrong spacing L55 doAssault.sqf
Copy link
Collaborator

@diwako diwako left a comment

Choose a reason for hiding this comment

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

Looks good. just a fair warining that checkVisibility is really slow and should not be used too much.

@jokoho48 jokoho48 added this to the 2.5.4 milestone Jun 8, 2022
@nk3nny nk3nny merged commit 0233b5b into master Jun 8, 2022
@nk3nny nk3nny deleted the Engagemt-fixes-with-lookAt-and-above-ground-suppression branch June 8, 2022 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants