Page MenuHomeWildfire Games

[gameplay] add visible garrison points to wall towers
Changes PlannedPublic

Authored by Nescio on May 26 2020, 10:29 PM.

Details

Reviewers
None
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Summary

Now placing units on walls no longer interferes with the opening and closing of gates (D1418/rP23710, rP23714, D2775/rP23731), it becomes possible to add visible garrison slots to wall tower as well, similar to medium and long wall segments. Advantages are that it looks better, is more realistic, and helps functionally differentiating wall towers, which are wall segments, from free standing towers (which are more expensive (100 wood + 100 stone), have a build limit of 30, and a mimum distance of 60 in between).

This patch:

  • Allows brit and gaul wall towers to be garrisoned, just like those of other civs.
  • Increases <GarrisonHolder/Max> from 2 to 4, because wall towers tend to be square. (For comparison, the smaller defense towers have 5, iber 8, and maur_tower_double 20.)
  • Moves the wall tower attack into a new template, template_structure_defensive_wall_tower_attack.xml, which is used by those civs (athen, brit, mace, maur, sele, spart) that have wall towers which have a closed room instead of an open platform on top; effectively this patch changes nothing for those.
  • Those that have a flat roof (cart, gaul, iber, kush, pers, ptol, rome) get four visible garrison points on top. Because garrisons are now visible, the garrison flags are removed from their actor files.
    • rome city wall and siege wall have visibly room for eight, but get only four, because they don't cost more than those of other civs.
    • cart gets eight slots instead of four, because it's twice as expensive, and visibly larger:
  • [EDIT]: The cost of the visibly garrisonable wall tower is reduced and made proportional to its health (cf. D2621/rP23713), whereas the ranged attack wall tower keeps its old cost.

See also D2477, D2760, D2783.

Test Plan

Check for mistakes and omissions. Agree this is an improvement.

Event Timeline

Nescio created this revision.May 26 2020, 10:29 PM
Owners added subscribers: Restricted Owners Package, Restricted Owners Package.May 26 2020, 10:29 PM

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/macos-differential/844/display/redirect

Nescio edited the summary of this revision. (Show Details)May 26 2020, 10:32 PM

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/2246/display/redirect

Nescio added inline comments.May 26 2020, 10:37 PM
binaries/data/mods/public/simulation/templates/structures/gaul_wall_tower.xml
6

<Y> + 1, like those of other civs.

binaries/data/mods/public/simulation/templates/template_structure_defensive_wall_tower_attack.xml
33

Necessary to keep maximum damage unchanged, since towers can now have a garrison of 4.

42

Raised from 60, because attack range is 72 + elevation, and vision range needs to be higher, because you can't attack enemies you can't see.

How it looks in game:







Angen added a subscriber: Angen.May 27 2020, 1:01 PM

I am for this change as well.

I garrison each unit in a tower, and he appears on the battlement at the top of the tower. They will be seen attacking enemies, and benefit from the tower’s elevation and defense.

But I think we need to apply this to all civilizations else some will get clear advantage with their towers. For that we need adjust art first.
Aside from that, units garrisoned on towers and walls should get some sort of armor bonus.
Also note (actually i don't mind): Javelins may or may not be able to attack because high difference, when trying to attack unit garrisoned on tower.

Aside from that, units garrisoned on towers and walls should get some sort of armor bonus.

Oops, my mistake, I should have inserted:

<Auras datatype="tokens">
  structures/wall_garrisoned
</Auras>

I assumed those lines were already present in template_structure_defensive_wall.xml, but apparently not. Also applies to D2760.

Also note (actually i don't mind): Javelins may or may not be able to attack because high difference, when trying to attack unit garrisoned on tower.

Sorry, I'm not sure I understand what you mean; could you elaborate? Javelineers have a range of 24 upwards.

But I think we need to apply this to all civilizations else some will get clear advantage with their towers. For that we need adjust art first.

As for art, athen, mace, spart share the same actor, which could use a redesign; sele has a different actor of similar design; maur needs its thin walls replaced with a gallery (also gate); brit needs its roof raised; there is a roofless variant present in its actor file which could be used temporarily, but that one looks a bit ugly.
It's also possible to have visible garrison slots inside, though I'm not sure that's a good idea:


Nescio added inline comments.May 27 2020, 4:57 PM
binaries/data/mods/public/simulation/templates/template_structure_defensive_wall_tower.xml
7–9

Also, it could be made cheaper here (and more expensive in the _attack template).
For comparison, a long wall segment costs only 28 stone and 45 time. With D2611, 36 and 36. If we want cost to be proportional to health for wall towers too, maybe make it 45 and 45?

Nescio added inline comments.May 27 2020, 4:59 PM
binaries/data/mods/public/simulation/templates/template_structure_defensive_wall_tower.xml
7–9

Correction: 36 × 4000 / 3000 = 48.

Nescio edited the summary of this revision. (Show Details)Jun 3 2020, 12:53 PM
Nescio added a reviewer: Restricted Owners Package.
Nescio removed subscribers: Restricted Owners Package, Restricted Owners Package.
Nescio updated this revision to Diff 12111.Jun 3 2020, 1:03 PM
Nescio edited the summary of this revision. (Show Details)
  • aura
  • cost
Owners added subscribers: Restricted Owners Package, Restricted Owners Package.Jun 3 2020, 1:03 PM
Vulcan added a comment.Jun 3 2020, 1:07 PM

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/2298/display/redirect

Nescio planned changes to this revision.Jul 20 2020, 1:01 PM