SiegeEngineers / aoe2techtree

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

Armor classes #106

Closed LOSER4ever closed 2 years ago

LOSER4ever commented 2 years ago

Hey! I am the writer of the Armor classes page on fandom wiki. I would like to draw your attention towards the following armor class major issues: All Buildings (except Port) - Port was a building in the Alpha and was never released, you can safely rename it to All Buildings Unique Units (except Turtle Ship) - Turtle Ship now have this armor class, so you should remove this Anti-Leitis - this armor class, while having values for units, is unused since the Leitis (since DoI) uses a new mechanic. No unit currently has an attack of this kind. I recommend you to remove it from stats of the units too and rename it accordingly. Cavalry - this is ambigous since it includes elephants but excludes Camels. I prefer renaming it to "Mounted Units except Camels" Ships & Camels & Sabs - better worded as "Ships except Fishing Ships" Rams - includes Armored Elephant line, Siege Towers and Unpacked Trebuchets Turtle Ships - includes Thirisadai now Armor class 0 - can safely name it Wonders, since I checked the Genie Editor and found only Wonder type buildings to have it Stone Defense - includes Harbors; excludes Castles and Kreposts Castles - includes Krepost Standard Buildings - armor ignoring effect against buildings; excludes Wonders

I have some minor issues as well: HD Camels - can safely drop "HD" from the name FE Gunpowder Units - can safely drop "FE" from the name War Elephants - can safely drop "War" from the name since all types of Elephants have this armor class Archers - "All Archers" preferred since it sometimes conveys that only Foot Archers have this class FE Predator Animals - can safely drop "FE" from the name

Apart from these, I also want to tell you that Poles do have Mills, it is just that Mills are automatically upgraded to Folwarks in Dark Age. You can represent the same using the upgrade lines.

HSZemi commented 2 years ago

Hi,

thanks for pointing out those issues. I have of course simply copied the list from AGE (with a few additions). But using more suitable names would of course be beneficial.

We will probably still display all attack and armor classes a unit has, even if some are not relevant for gameplay currently.

The Folwark placement is mainly a space issue, since we only have two rows per age, but would need three to put the Farm below Mill and Folwark like you suggested.

LOSER4ever commented 2 years ago

Hey, thanks for a quick reply. I totally understand the 2 row limitation, so I see why you can't show Folwark as an upgrade to Mill, but I think you will definitely make Mills available for Poles.

About Armor Class 31 Anti-Leitis, I suggest you use a suitable name like deprecated or obsolete.

HSZemi commented 2 years ago

Since you can never actually build a mill as the Poles (?), showing the Mill as available next to the Folwark seems misleading in my opinion.

How's this for the class names? https://github.com/SiegeEngineers/aoe2techtree/blob/6a72ed378770b6f15e0280309b4951bd4db0cc8b/js/main.js#L9-L48

LOSER4ever commented 2 years ago

Hey, thanks for editing. I notice you copied the names for the wiki, which I appreciate. You have some minor changes too, which I appreciate and am planning to incorporate. I went too technical at some places, which I shall correct.

8: '<abbr title="(except Camels)">Mounted units</abbr>' - I prefer this because Cavalry usually denotes mounted melee units. Mounted units is a broader term which included Cavalry Archers, Missionaries, etc
14: 'Wolves' should be fine enough
17: 'Anti-Wall Non-Gunpowder Siege' - too complex to understand at first glance, would prefer "Battering Rams, Elephant Rams, Siege Towers and unpacked Trebuchets", or if you prefer "High pierce armor Siege weapons". Note that packed Trebuchets are not included.
24: 'Boars' should be fine enough
28: 'Cavalry Archers' - this is fine too, Mounted Archers works too, not really a problem
31: '<abbr title="(obsolete)">Anti-Leitis</abbr>'
32: 'Condottieros' - Condottieri
33: '<abbr title="(no unit has this armor class)">Gunpowder units secondary projectile attack</abbr>'
HSZemi commented 2 years ago

Are there other relevant gunpowder units apart from Organ Guns for which 33 is relevant?

LOSER4ever commented 2 years ago

Hussite Wagons, they use the same secondary projectile, hence also do 1002 damage against armor class 33, or put simply 2 damage (irrespective of armor) to all units it hits.

LOSER4ever commented 2 years ago

Giving any unit a +x attack bonus vs an unused armor class basically gives it +(x-1000) attack bonus vs all units. The same concept is used for Organ Guns and Hussite Wagon attacks with armor class 33.

I made a fair generalisation, because there are actually some units which receive the full +x attack bonus. These units are sheep (herdable animals), hero wolves among others.

HSZemi commented 2 years ago

oki

LOSER4ever commented 2 years ago

If you are writing the stats, and not embedding from the dataset, you can write directly for Hussite Wagons and Organ Guns. Secondary projectiles deal 2 damage vs everything, so that it doesn't become complex.

HSZemi commented 2 years ago

The changes are now in a2332168b692e5a9eb8d6821ea72f638d6d6636d. Thanks for your input :yellow_heart: