From 0499ded1b3a94b9c850122a27fe1820e9a76be74 Mon Sep 17 00:00:00 2001 From: Dan Albert Date: Sat, 19 Nov 2022 12:38:23 -0800 Subject: [PATCH] Clarify `AirAssault` flight plan method name. `engagement_distance` is used elsewhere to mean commit distance, so this looked like a bug when I stumbled across it. Rename it to be more explicit. --- game/ato/flightplans/airassault.py | 3 +-- game/missiongenerator/logisticsgenerator.py | 2 +- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/game/ato/flightplans/airassault.py b/game/ato/flightplans/airassault.py index a5b2972a..8df43e57 100644 --- a/game/ato/flightplans/airassault.py +++ b/game/ato/flightplans/airassault.py @@ -63,8 +63,7 @@ class AirAssaultFlightPlan(StandardFlightPlan[AirAssaultLayout]): return None @property - def engagement_distance(self) -> Distance: - # The radius of the WaypointZone created at the target location + def ctld_target_zone_radius(self) -> Distance: return meters(2500) @property diff --git a/game/missiongenerator/logisticsgenerator.py b/game/missiongenerator/logisticsgenerator.py index 77a0ab7f..6cfab31c 100644 --- a/game/missiongenerator/logisticsgenerator.py +++ b/game/missiongenerator/logisticsgenerator.py @@ -46,7 +46,7 @@ class LogisticsGenerator: target_zone = f"{self.group.name}TARGET_ZONE" self.mission.triggers.add_triggerzone( self.flight.flight_plan.layout.target.position, - self.flight.flight_plan.engagement_distance.meters, + self.flight.flight_plan.ctld_target_zone_radius.meters, False, target_zone, )