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

Adds teleport option to taskPatrol #428

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

nk3nny
Copy link
Owner

@nk3nny nk3nny commented Nov 23, 2024

Extends Modules (Zeus and Eden) with a teleport option in taskPatrol. When enabled, it functions much like Garrisons-- placing the group at at a randomly selected position with the patrol area.

@nk3nny nk3nny requested review from jokoho48 and diwako November 23, 2024 17:40
@nk3nny nk3nny self-assigned this Nov 23, 2024
@@ -34,13 +34,14 @@ switch (_mode) do {
[LSTRING(Module_TaskPatrol_Range_DisplayName), "SLIDER", LSTRING(Module_TaskPatrol_Range_ToolTip), [20, 2000], [1, 0.5], TASK_PATROL_SIZE, 1],
[LSTRING(Module_TaskPatrol_Waypoints_DisplayName), "SLIDER", LSTRING(Module_TaskPatrol_Waypoints_ToolTip), [2, 15], [2, 1], TASK_PATROL_WAYPOINTCOUNT, 0],
[LSTRING(Module_TaskPatrol_MoveWaypoints_DisplayName), "BOOLEAN", LSTRING(Module_TaskPatrol_MoveWaypoints_ToolTip), TASK_PATROL_MOVEWAYPOINTS],
[LSTRING(Module_Task_EnableReinforcement_DisplayName), "BOOLEAN", LSTRING(Module_Task_EnableReinforcement_ToolTip), TASK_PATROL_ENABLEREINFORCEMENT]
[LSTRING(Module_Task_EnableReinforcement_DisplayName), "BOOLEAN", LSTRING(Module_Task_EnableReinforcement_ToolTip), TASK_PATROL_ENABLEREINFORCEMENT],
[LSTRING(Module_TaskGarrison_Teleport_DisplayName), "BOOLEAN", LSTRING(Module_TaskGarrison_Teleport_Tooltip), TASK_PATROL_TELEPORT]
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When we use it in multiple places, we should remove the Garrison from the name

@@ -12,6 +12,7 @@
* 4: Area the AI Camps in, default [] <ARRAY>
* 5: Dynamic patrol pattern, default false <BOOL>
* 6: enable dynamic reinforcement <BOOL>
* 7: Teleport group <BOOL>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This needs far better wording. The comment for the associated code says it teleports the group to a random waypoint position, which itself is confusing cuz it means a random waypoint from the ones the group currently has.

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

Successfully merging this pull request may close these issues.

3 participants