Opened 5 years ago

Closed 3 years ago

#5593 closed enhancement (fixed)

Unify UnitAI and "AnimalAI".

Reported by: Freagarach Owned by: Freagarach
Priority: Should Have Milestone: Alpha 25
Component: Simulation Keywords:
Cc: Patch: ​Phab:D2646

Description

There are some tickets about AnimalAI (#563, #1724, #3919), some suggest splitting the behaviour of animals from UnitAI. That would lead to much duplication and nigh any benefit.

IMHO we should rather unify the two, letting animals be able to behave exactly the same as "normal" units and vice versa (e.g. humans wandering).

Change History (5)

comment:1 by Freagarach, 4 years ago

Patch: → Phab:D2646

comment:2 by Silier, 3 years ago

Milestone: Backlog → Work In Progress

comment:3 by Silier, 3 years ago

Owner: set to Freagarach

comment:4 by Freagarach, 3 years ago

Milestone: Work In Progress → Alpha 25
Priority: Nice to Have → Should Have

comment:5 by Freagarach, 3 years ago

Resolution: → fixed
Status: new → closed

In 24953:

Unify UnitAI and AnimalAI.

r8995 / rP8995 (#563) introduced an ANIMAL-state from a separate AnimalAI-component (introduced in r7763 / rP7763).
This patch merges that separate state and brings the ROAMING and FEEDING (renamed to LINGERING) under the INDIVIDUAL.IDLE-state.

This enables e.g. city-building mods to have human units that linger and roam or animals that behave like humans.

The specific values for animals might need tweaking after this.

Differential revision: D2646
Fixes: #1832, #5593
Comments by: @Angen, @Langbart, @Nescio, @Stan, @wraitii
Refs.: #3919

Note: See TracTickets for help on using tickets.