Skip to content

Feat: Rewrite SAM targeting into a coordinator / orchestrator class #5601

Description

@JB940

Is your feature request related to a problem? Please describe.
All SAMs currently calculate separately to intercept nukes. While they do do checks whether a missile is already being intercepted, there is no way to organize which SAM should target which. There is also a big performance cost to doing this for each SAM individually

Describe the solution you'd like
An orchestrating or coordinater class that takes the responsibility of all of this, it runs on each missile and calculates which SAM should intercept it, so individual SAMs never have to look.

This allows for smarter targeting. IE spreading the missiles between SAMs instead of first SAM runs out of ammo and gets blown up for nukes flying over it

This also has potentially big performance gains, SAM logic is a pretty intensive part of the tick calcs when everyone builds dozens and dozens, and hundreds of warheads fly about.

Describe alternatives you've considered
Perf buffing SAMs and nukes. We've implemented these, but it can still be better. Targeting can't really be improved without cross communicating in some shape and this is the cleanest method.

Assignment

  • I'd like to be assigned to this issue and work on it myself

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

approvedApproved for a PR, if you assigned to the issue.

Type

No type

Projects

Relationships

None yet

Development

No branches or pull requests

Issue actions