Page MenuHomeWildfire Games

[gameplay] detower outpost
ClosedPublic

Authored by Nescio on Sep 3 2020, 2:01 PM.

Details

Summary

This patch is a follow-up to D2933/rP24016. It does the following:

  • Removes the tower-like <Attack> and <BuildingAI> from the outpost, and instead gives it a <TurretHolder> (i.e. visible garrison slot) and <Auras>, like walls.
  • Corrects footprint, rubble, obstruction, status bar, and foundation sizes of the outpost template.
  • Because outposts are now differentiated from towers, they no longer need to inherit from template_structure_defensive_tower.xml, therefore *_outpost.xml is renamed to template_structure_defensive_outpost.xml.
  • Updates all specific outpost templates accordingly.

How it looks in game:

Test Plan

Check for mistakes and omissions. Agree with the proposed changes.

Event Timeline

Nescio created this revision.Sep 3 2020, 2:01 PM
Owners added subscribers: Restricted Owners Package, Restricted Owners Package.Sep 3 2020, 2:01 PM
Nescio added inline comments.Sep 3 2020, 2:04 PM
binaries/data/mods/public/simulation/templates/template_structure_defensive_outpost.xml
88

This is probably the most controversial change: like walls, ungarrisoned outposts don't reveal their surroundings, whereas garrisoned units get extensive vision.

binaries/data/mods/public/simulation/templates/template_structure_defensive_tower.xml
62–63

Already defined in template_structure.xml.

Nescio requested review of this revision.Sep 3 2020, 2:06 PM

I'm very much in favour of this. Just keep in mind that normal infantry won't be able to defend your outpost anymore (no swordsmen shooting arrows).

Nescio added a comment.Sep 3 2020, 2:20 PM

I'm very much in favour of this.

Thanks, I'm glad you like it.

Just keep in mind that normal infantry won't be able to defend your outpost anymore (no swordsmen shooting arrows).

Yes, I'm aware of that; it's intentional. While they can't attack, garrisoned melee infantry is safe from enemy melee attacks, and does get additional resistance and extensive vision.

Stan added a subscriber: Stan.Sep 3 2020, 2:22 PM

Could be nice to keep the flags on just in case somebody has sillouetes off.

In D2986#130520, @Stan wrote:

Could be nice to keep the flags on just in case somebody has sillouetes off.

+1 (Also looks nice.)

Nescio updated this revision to Diff 13377.Sep 3 2020, 2:30 PM
Nescio edited the summary of this revision. (Show Details)
Nescio removed subscribers: Restricted Owners Package, bb.
Freagarach added a reviewer: Freagarach.EditedSep 3 2020, 2:31 PM

Thanks, I'm glad you like it.

Yeah, these kind of changes (like D2783 and D2760) really improve the SP experience, IMHO.
Now wait for the green light of the balancing department.

like D2783 and D2760

If I recall correctly, @badosu, @borg-, and @genava55 were in favour, so those two are basically waiting for a team member to review and commit them.

D2783 was only accepted by one member of the Balancing group, hence I asked for more feedback there.
D2760 has an open question.

wraitii added a subscriber: wraitii.Sep 3 2020, 2:55 PM

This seems rather trivially a good change to me.

Nescio added a comment.Sep 3 2020, 3:26 PM

By the way, the tooltip (gui/text/tips/outposts.txt + art/textures/ui/loading/tips/outposts.png) is already outdated and not very informative, therefore I'm inclined to delete it; objections?

Or improve them?

Nescio updated this revision to Diff 13378.Sep 3 2020, 3:33 PM
  • update gui/text/tips/outposts.txt
Owners added a subscriber: Restricted Owners Package.Sep 3 2020, 3:33 PM
Freagarach added inline comments.Sep 3 2020, 3:36 PM
binaries/data/mods/public/gui/text/tips/outposts.txt
5 ↗(On Diff #13378)

It still decays, right? So perhaps keep that information?

Nescio added inline comments.Sep 3 2020, 3:39 PM
binaries/data/mods/public/gui/text/tips/outposts.txt
5 ↗(On Diff #13378)

Yes, but that's not unique, other structures have territory decay too.

Freagarach added inline comments.Sep 3 2020, 3:44 PM
binaries/data/mods/public/gui/text/tips/outposts.txt
5 ↗(On Diff #13378)

I understand, but most other structures are not buildable in neutral territory, these are tips for inexperienced people. Hence I'd suggest to add and to keep the outpost from decaying in neutral territory (or something similar) to the last sentence.

borg- added a comment.Sep 3 2020, 5:59 PM

I really like this

Nescio updated this revision to Diff 13382.Sep 3 2020, 6:28 PM
  • rephrase gui tip
binaries/data/mods/public/simulation/templates/template_structure_defensive_outpost.xml
62–68

Already present in parent.

In D2986#130577, @borg- wrote:

I really like this

(You *may* click accept ;) )

Freagarach accepted this revision.Sep 3 2020, 6:39 PM
  • Template values are correct.
  • Looks very nice in game (much better than an open structure where the garrisoned entity disappears).
  • Adjusted dimensions make sense and look good.
This revision is now accepted and ready to land.Sep 3 2020, 6:39 PM
borg- added a comment.EditedSep 3 2020, 7:04 PM

Following the idea of the patch, outpost could be just a neutral building, cannot be captured, just destroyed or garrison. Can only be built in neutral territory.

While such a neutral structure would indeed be nice, that is not supported (yet).

borg- accepted this revision.Sep 4 2020, 12:00 PM
This revision was automatically updated to reflect the committed changes.