Opened 4 years ago

Closed 3 years ago

#4352 closed defect (fixed)

Relative templates for walk/run speed

Reported by: elexis Owned by:
Priority: Nice to Have Milestone: Alpha 23
Component: UI & Simulation Keywords:
Cc: Patch:

Description (last modified by fatherbushido)

http://trac.wildfiregames.com/changeset/18025/ps/trunk/binaries/data/mods/public/simulation/templates/units/gaul_champion_fanatic.xml added 34 run-speed for naked fanatics, which is faster than everything else while it should be slightly below the cavalry walk-speed probably.

Furthermore slow walking spearmen infantry champions have a much higher run speed than the fast walking swordsmen infantry champions.

These inconsistencies should be cleaned, future mistakes could be reduced by using relative templates (op=mul and op=add).

Attachments (1)

Sans nom 1.png (31.5 KB) - added by fatherbushido 3 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 3 years ago by fatherbushido

comment:2 Changed 3 years ago by fatherbushido

In 20230:

Use relative templates for unit speed. No significant value change. It makes balancing easier and will permit to change the speed of all units by changing one number. Refs #4352.

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

Changed 3 years ago by fatherbushido

Attachment: Sans nom 1.png added

comment:3 Changed 3 years ago by fatherbushido

Description: modified (diff)

I think we can close that. A next step would be to adress the inconsistency of current values:

  • fixing the cs - champ - hero consistency (no opinion from my side, perhaps making all equal)
  • fixing the cav - inf consistency (keeping more or less the same ratio for cs, champ, hero)
  • fixing the internal consistency (no opinion from my side, light units are intended to be faster, but that would make ranged units too powerfull...)

So the balance guy should take care of that.

http://trac.wildfiregames.com/raw-attachment/ticket/4352/Sans%20nom%201.png

Last edited 3 years ago by fatherbushido (previous) (diff)

comment:4 Changed 3 years ago by elexis

Keywords: simple removed
Milestone: BacklogAlpha 23
Resolution: fixed
Status: newclosed

Thanks for the patch!

As you pointed out, there are some inconsistencies, but probably no need to have an open ticket for that, so agree to close it.

Note: See TracTickets for help on using tickets.