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). A few seconds after loading the game, the unit with entity ID '1522' will begin sliding. (The unit is a ballista in the north-eastern part of the map.)

Version 1, edited 11 years ago by zoot (previous) (next) (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.