Opened 7 years ago

Last modified 4 years ago

#4028 new enhancement

Replace the elevation bonus for units with something less resource intensive.

Reported by: sanderd17 Owned by:
Priority: Should Have Milestone: Backlog
Component: Simulation Keywords:
Cc: Patch:

Description (last modified by elexis)

The elevation bonus is great for buildings: the higher buildings are placed, the further they can shoot.

But it gives some problems for units: units don't know what their range is until they are at their possible firing location. Which means they possibly have to repath to a better location.

It would be better to replace it with a sort of static attack or armour stat advantage based on elevation difference.

Hint: in UnitAI replace the CheckTargetAttackRange and MoveToTargetAttackRange calls to CheckTargetRange and MoveToTargetRange calls.

Change History (6)

comment:2 by fatherbushido, 6 years ago

We can perhaps keep the range bonus when attacking but don't take care of it when moving (I mean don't try to move at that max elevated range).

comment:3 by wraitii, 6 years ago

This is going to be a little irrelevant when my unitMotion branch gets merged, since I will change UnitAI to work on a "distance check" and unitMotion will automatically try to get in the correct range. Will probably call the pathfinder a little less (and it's inevitable to an extent anyway).

comment:4 by fatherbushido, 6 years ago

wraitii: I thought of that while reading your posts in forum :)

comment:5 by elexis, 5 years ago

Description: modified (diff)
Keywords: simple removed

Something related at Phab:D781 Removing the simple keyword as it needs some design decision

comment:6 by Imarok, 4 years ago

Component: UI & SimulationSimulation

Move tickets to Simulation as UI & Simulation got some sub components.

Note: See TracTickets for help on using tickets.