Opened 9 years ago

Closed 9 years ago

Last modified 6 years ago

#3474 closed defect (fixed)

Animal roaming

Reported by: elexis Owned by: mimo
Priority: Must Have Milestone: Alpha 19
Component: Core engine Keywords:
Cc: Itms Patch:

Description (last modified by elexis)

Animals only walk some steps when roaming and rotate too frequently, almost looks like spinning. Similar problems were encountered in #1833. Bug was introduced in r17028.

https://trac.wildfiregames.com/raw-attachment/ticket/3474/deer_ballet.gif

https://trac.wildfiregames.com/raw-attachment/ticket/3474/whale_ballet.gif

Attachments (4)

deer_ballet.webm (1.2 MB ) - added by elexis 9 years ago.
Sample video
whale_ballet.gif (606.9 KB ) - added by elexis 9 years ago.
deer_ballet.gif (1.5 MB ) - added by elexis 9 years ago.
Same.
ticket3474.patch (678 bytes ) - added by mimo 9 years ago.

Change History (13)

comment:1 by mimo, 9 years ago

It's a different problem than in #1833. Here the animal seems to turn towards the random direction it should move, but the move stops immediately, so it chooses a new random direction, faces towards it , ... and so on during all its roaming time. I think it is linked to #3405 as both flee and roaming uses MoveToPointRange which seems to be broken when the unit is inside the range it should move to. If this behaviour is recent (i.e. not introduced with the new pathFinder), it may be because of r17013

by elexis, 9 years ago

Attachment: deer_ballet.webm added

Sample video

comment:2 by elexis, 9 years ago

Reproduce: Start any map with roaming animals, like the default skirmish map. Notice whales are also affected. Caused by: r17028, the revisions before that don't have it, but that revision and every after that does.

comment:3 by elexis, 9 years ago

Description: modified (diff)

by elexis, 9 years ago

Attachment: whale_ballet.gif added

by elexis, 9 years ago

Attachment: deer_ballet.gif added

Same.

comment:4 by elexis, 9 years ago

Description: modified (diff)

Can we assume that an error in MoveToPointRange also causes errors for the players units?

comment:5 by elexis, 9 years ago

Milestone: BacklogAlpha 19
Priority: Should HaveMust Have

Setting a19 for awareness of the current pathfinder bugs. Might be pushed later if the bug doesn't have any side-effects on the units too.

by mimo, 9 years ago

Attachment: ticket3474.patch added

comment:6 by mimo, 9 years ago

Keywords: reviev added; pathfinding removed

The attached patch should fix the bug.

comment:7 by mimo, 9 years ago

Owner: set to mimo
Resolution: fixed
Status: newclosed

In 17124:

fix typo in UnitMotion, fixes #3474

comment:8 by mimo, 9 years ago

Keywords: reviev removed

comment:9 by elexis, 6 years ago

Description: modified (diff)
Note: See TracTickets for help on using tickets.