HomeWildfire Games

Use new FastMoving class instead of Cavalry in AI/petra.
AuditedrP23916

Description

Use new FastMoving class instead of Cavalry in AI/petra.

This class is an AI hint that such units are to be treated as moving fast, e.g. war dogs, or cavalry.
This makes it easier to introduce camels and chariots correctly.

Further work is required to make the AI unit choices less hardcoded.

Patch by: Nescio

Reviewed By: Angen, wraitii

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

Event Timeline

Freagarach raised a concern with this commit.Dec 24 2020, 6:41 AM
Freagarach added a subscriber: Freagarach.
ERROR: JavaScript error: simulation/ai/petra/worker.js line 115 ent.isFastMoving is not a function PETRA.Worker.prototype.update@simulation/ai/petra/worker.js:115:32 PETRA.BaseManager.prototype.update@simulation/ai/petra/baseManager.js:1064:21 PETRA.HQ.prototype.update@simulation/ai/petra/headquarters.js:2764:54 PETRA.PetraBot.prototype.OnUpdate@simulation/ai/petra/_petrabot.js:118:11 m.BaseAI.prototype.HandleMessage@simulation/ai/common-api/baseAI.js:64:7
/ps/trunk/binaries/data/mods/public/simulation/ai/petra/worker.js
115

PETRA.isFastMoving(ent)

This commit now has outstanding concerns.Dec 24 2020, 6:41 AM
All concerns with this commit have now been addressed.Dec 26 2020, 8:16 AM