Diffusion 0 A.D. Public Repository rP22446

UnitMotion / AI - remove the special 'move' animation, make UnitMotion inform…
rP22446
Actions

Description

UnitMotion / AI - remove the special 'move' animation, make UnitMotion inform the visual actor directly.

Use UpdateMovementState to inform the visual actor of the unit's speed, which cill update the movement animation accordingly.
The removes the need for UnitAI to handle movement animation using the special "move" state.

Differential Revision: https://code.wildfiregames.com/D1901

Details

Committed

wraitii

Jul 9 2019, 9:56 PM

Differential Revision

D1901: UnitMotion / AI - remove the special walk animation and let unitMotion take charge

Parents

rP22445: Adds an tool to pick water high by terrain click in Atlas.

Branches

Unknown

Tags

Unknown

Build Status

Buildable 8267
Build 13484: Trigger Windows Autobuild
Build 13483: Post-Commit Build	Jenkins

Event Timeline

wraitii committed rP22446: UnitMotion / AI - remove the special 'move' animation, make UnitMotion inform….Jul 9 2019, 9:56 PM

Harbormaster completed building B8267: rP22446: UnitMotion / AI - remove the special 'move' animation, make UnitMotion inform….Jul 9 2019, 10:07 PM

Silier mentioned this in D2438: Fix formationmember not leaving walking state if offset cannot be reached.Nov 23 2019, 1:41 PM

Silier mentioned this in rP23496: Fix formationmember not leaving walking state if offset cannot be reached.Feb 13 2020, 8:44 PM

Silier added a subscriber: Silier.Jun 5 2020, 3:59 PM

Silier added inline comments.

/ps/trunk/binaries/data/mods/public/simulation/components/UnitAI.js

3063

It would be here.
Reported by elexis:

`<elexis> wraitii Freagarach and whomever worked with UnitAI, there is a small regression in svn compared to a23b: when changing perspective to gaia and sending a move order to a non-domestic gaia animal, the animal will have an animation-reset and not perform any order, whereas in a23b it only didnt perform any order without animation reset. perhaps you can identify the commit
`

Freagarach mentioned this in D1960: Move the controllability of an entity to cmpIdentity..Jun 5 2020, 7:42 PM

Freagarach mentioned this in rP24148: Move the controllability of an entity to cmpIdentity..Nov 9 2020, 9:38 AM

wraitii mentioned this in D3221: Fix UnitMotion updating the visual Actor with the target speed, not the real speed..Dec 14 2020, 6:48 PM

wraitii mentioned this in rP24396: Fix UnitMotion updating the visual Actor with the target speed, not the real….Dec 15 2020, 8:44 AM

wraitii mentioned this in D3619: Fix a rare case of unit sliding..Mar 1 2021, 5:29 PM

wraitii mentioned this in rP25011: Fix a rare case of unit 'gliding' while moving..Mar 4 2021, 7:33 PM