Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#5761 closed defect (fixed)

Units refuse to enter formation

Reported by: elexis Owned by: Silier
Priority: Should Have Milestone: Alpha 24
Component: Simulation Keywords:
Cc: Patch: Phab:D2761

Description (last modified by elexis)

In this r23670 replay units received a formation order, then move orders. Then new units were trained, selected along with the existing formation. The GUI said they were not in formation, clicking on the formation did not make them enter formation. This happens at minute 1 in the replay.

I didn't have time to isolate the bug, hopefully it's a user error.

Attachments (1)

commands.txt (18.5 KB ) - added by elexis 4 years ago.

Download all attachments as: .zip

Change History (6)

by elexis, 4 years ago

Attachment: commands.txt added

comment:1 by elexis, 4 years ago

Description: modified (diff)

comment:2 by Freagarach, 4 years ago

So far, I have narrowed this down to something I wanted to report earlier. What happens is that a formation is trying to be made at turn 290, but there is one entity (entity 168, good luck visually finding that one, it is hidden underneath entity 181) that is not included in the same cluster as the rest. This causes the formation to disband, since the one entity cannot be set to a formation and thus both clusters can't be set.

This means two things are noticed herein:

  • Why does Command.txt/ClusterEntities exclude that one entity from the main cluster? <- A real bug.
  • When having one cluster large enough to form a formation, and another one which can't, the user expects the one that can to actually form the formation. (This can be tested easier by having two units close by and one far of (at least 60 m), selecting all three units and trying to set them to a formation.

comment:3 by Silier, 4 years ago

Milestone: BacklogWork In Progress
Owner: set to Silier
Patch: Phab:D2761

comment:4 by Silier, 4 years ago

Resolution: fixed
Status: newclosed

In 23693:

Fix computation of clusters and var->let

Compare matrix values against undefined instead of relying on || because || does not work as intended when distance is 0.

Differential Revision: https://code.wildfiregames.com/D2761
Fixes: #5761

comment:5 by Silier, 4 years ago

Milestone: Work In ProgressAlpha 24
Note: See TracTickets for help on using tickets.