Paths

Table of Contentst

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

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

Details

Reviewers

Group Reviewers

Restricted Owners Package	(Owns No Changed Paths)
Balancing

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.

Diff Detail

Repository

rP 0 A.D. Public Repository

Branch

/ps/trunk

Lint

Lint OK

Unit

No Unit Test Coverage

Build Status

Buildable 11917
Build 22326: Vulcan Build	Jenkins
Build 22325: Vulcan Build (Windows)	Jenkins
Build 22324: arc lint + arc unit

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

Nescio mentioned this in D1418: Enable garrisoning on gates / fix movement of units with visible garrison points.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

Harbormaster failed remote builds in B11835: Diff 12031!May 26 2020, 10:30 PM

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:

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.

Harbormaster completed remote builds in B11835: Diff 12031.May 29 2020, 11:10 AM

For that we need adjust art first.

https://wildfiregames.com/forum/index.php?/topic/28262-request-improve-some-wall-towers/

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.

aura
cost

Owners added subscribers: Restricted Owners Package, Restricted Owners Package.Jun 3 2020, 1:03 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

Harbormaster completed remote builds in B11917: Diff 12111.Jun 3 2020, 1:07 PM

Nescio mentioned this in D2783: [gameplay] allow a few more units on medium and long walls.Jun 3 2020, 1:54 PM

Nescio mentioned this in D2803: [gameplay] reduce palisade health and repair time ratio.Jun 8 2020, 10:49 AM

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

Nescio mentioned this in D2477: [gameplay] garrison units on short wall segments.Jul 21 2020, 1:08 PM

Nescio mentioned this in D2760: [gameplay] add visible garrison points to gates.

Nescio edited the summary of this revision. (Show Details)

Nescio mentioned this in D2993: [gameplay] add visible garrison points to rome siege wall tower.Sep 4 2020, 9:26 PM

(Please note that wall towers are also refuge positions for women now.)

Herald added a reviewer: Balancing. · View Herald TranscriptOct 8 2020, 5:53 PM

Freagarach mentioned this in D3466: [gameplay] Make wall turrets upgrade into proper towers..Jan 24 2021, 9:32 PM

I failed to realised that this existed, but I fully agree with this and duplicate it somewhat in D3466.

Based on that plan, I think you can simply handle changing wall turrets to:

only handle visible garrisoning (no other garrisoning, except possibly allowing women if we want to keep that functionality)
make them slightly better, and slightly more expensive than small wall pieces, but not more than medium (in D3466 I did 16 stone/ 16 time / 1000 HP).
- it makes no sense that towers have so much HP, in reality they would arguably be weaker than wall pieces, not stronger
possibly reduce the visible garrisoning # a bit: it looks very crowded as you've done it (archers would basically get in each other's way), I'd go for 2 or 3 for regular walls, possibly 5-6 for cart.
remove the tower_attack class, we'll handle that in D3466