-
Notifications
You must be signed in to change notification settings - Fork 328
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
base: master
Are you sure you want to change the base?
Add new target category, add it to auto trajectory units #4162
Conversation
For reference the BARb AI configures target categories in link below. Don't forget to update the AI behavior when adding armordefs or categories.
|
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. |
Basing it exclusively off of speed does not account for how maneuverable a unit is. |
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 |
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.