SiegeEngineers / aoe2techtree

Age of Empires II Tech Tree
https://aoe2techtree.net/
MIT License
225 stars 59 forks source link

Added exceptions for showing irrelevant values #111

Open LOSER4ever opened 2 years ago

LOSER4ever commented 2 years ago

Removed Armor class 31 from armors of units since Obsolete. Removed 0 attacks of units (like archers having 0 vs Rams, etc), but made sure that armor classes having units with negative armor are rendered properly. Exceptions are hardcoded - may need to be updated with time. Current exceptions: War Elephants, Cavalry and Siege Weapons since Ballista Elephants have -2 armor in each. Mounted Archers since Elephant Archers have -7 armor. Base melee since Rams have negative armor.

HSZemi commented 2 years ago

Hmm that is a lot of hardcoding based on the current balance…

LOSER4ever commented 2 years ago

So what did you think of it?

HSZemi commented 2 years ago

I think I prefer to keep it as it currently is

paulirish commented 2 years ago

@LOSER4ever is it possible to author this without the class ID hardcoding?

LOSER4ever commented 1 year ago

You need to check all armor classes across units to see if the unit has negative armor.