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

Merged

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.

@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.

@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.

@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
@SethDGamre SethDGamre marked this pull request as ready for review February 6, 2025 20:28
@SethDGamre
Copy link
Collaborator Author

now the target category is used only for high trajectory which would really struggle to hit the quick units.

@WatchTheFort WatchTheFort merged commit f78fc34 into beyond-all-reason:master Feb 13, 2025
kroIya added a commit to kroIya/Beyond-All-Reason that referenced this pull request Feb 19, 2025
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.

None yet

6 participants