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 new target category, add it to auto trajectory units #4162

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

SethDGamre
Copy link
Collaborator

I noticed that groundscouts aren't the only units fast enough to give auto trajectory units a hard time. This PR creates a new target category for units that are faster than a rez bot and gives it to those auto trajectory units for starters.

https://youtu.be/qe8-Pg3lzwg

Testing steps:

Spawn any of the units changed in this PR and test it on various targets greater than and less than 78 movement speed and that moves around on the surface -- this is what qualifies them for FASTSURFACE category.

gamedata/alldefs_post.lua Outdated Show resolved Hide resolved
@S3KCentrifugal
Copy link
Contributor

S3KCentrifugal commented Jan 13, 2025

For reference the BARb AI configures target categories in link below. Don't forget to update the AI behavior when adding armordefs or categories.

gamedata/alldefs_post.lua Outdated Show resolved Hide resolved
@SethDGamre
Copy link
Collaborator Author

Beyond-All-Reason/luarules/configs/BARb/stable/script/common.as

The categories listed there seem to be for identifying the movement type of the unit - land, sea, air. The new category is applied in addition to those and unless there's a portion of the AI code that needs to know if a unit moves fast or not, this isn't yet useful there.

gamedata/alldefs_post.lua Outdated Show resolved Hide resolved
@kroIya
Copy link
Contributor

kroIya commented Jan 24, 2025

Basing it exclusively off of speed does not account for how maneuverable a unit is.
corpship is fast at 82.5 speed, but is large and turns slowly, and is an easy target to hit. Navy is commonly defended against with static artillery

@SethDGamre
Copy link
Collaborator Author

Basing it exclusively off of speed does not account for how maneuverable a unit is. corpship is fast at 82.5 speed, but is large and turns slowly, and is an easy target to hit. Navy is commonly defended against with static artillery

Maneuverability is a better metric. Any insights into how I can filter by manueverability and which units would be slippery?

Perhaps this category should become SLIPPERYSURFACE

@SethDGamre SethDGamre marked this pull request as draft January 24, 2025 16:31
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.

6 participants