Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#1823 closed defect (fixed)

Units in formation sometimes appear to slide

Reported by: zoot Owned by: leper
Priority: Should Have Milestone: Alpha 13
Component: UI & Simulation Keywords:
Cc: Patch:

Description

Sometimes, one or more units in a formation fail to play its walking animation when moving:

http://i.imgur.com/IyVY5.gif

See also: http://www.wildfiregames.com/forum/index.php?showtopic=16969

Attachments (2)

unitslide.tar.gz (28.1 KB ) - added by zoot 11 years ago.
Savegame (and map) with a 'sliding' formation member.
sliding2.0adsave (101.9 KB ) - added by zoot 11 years ago.

Download all attachments as: .zip

Change History (8)

by zoot, 11 years ago

Attachment: unitslide.tar.gz added

Savegame (and map) with a 'sliding' formation member.

comment:1 by zoot, 11 years ago

Attached above (unitslide.tar.gz) is a savegame (and its corresponding map) in which a unit will begin 'sliding' a few seconds after loading. The unit has entity ID '46'.

Just before the unit begins sliding, OnMotionChanged is invoked on its UnitAI, and this subsequently leads to the unit entering a "FORMATIONMEMBER.IDLE" state prematurely (while it is still moving), which in turn disables the walking animation.

Why OnMotionChanged is invoked, I don't know, but it appears to be due to some kind of failure in UnitMotion because of the unit's path 'colliding' with a house.

Last edited 11 years ago by zoot (previous) (diff)

comment:2 by leper, 11 years ago

Owner: set to leper
Resolution: fixed
Status: newclosed

In 13079:

Stop formation members from sliding. Fixes #1823. Thanks to zoot for the test case!

comment:3 by leper, 11 years ago

Component: Core engineUI & Simulation
Milestone: BacklogAlpha 13

Thanks for the great test case!

by zoot, 11 years ago

Attachment: sliding2.0adsave added

comment:4 by zoot, 11 years ago

Resolution: fixed
Status: closedreopened

The first case is fixed for me too, but apparently here is another one (sliding2.0adsave). About 15 seconds after loading the game, the unit with entity ID '1522' will begin sliding. (The unit is a packed ballista in the north-eastern part of the map.)

Last edited 11 years ago by zoot (previous) (diff)

comment:5 by leper, 11 years ago

Resolution: fixed
Status: reopenedclosed

In 13112:

Only call FinishOrder on MoveCompleted if we didn't get an error. Fix #1823.

in reply to:  4 comment:6 by leper, 11 years ago

Replying to zoot:

The first case is fixed for me too, but apparently here is another one (sliding2.0adsave).

Thanks for providing another test case! I overlooked another case that triggered the OnMotionChanged hook, but it should be fixed now.

Note: See TracTickets for help on using tickets.