Page MenuHomeWildfire Games

[gameplay] delete wall technology
ClosedPublic

Authored by Nescio on Thu, Jul 16, 3:27 PM.

Details

Reviewers
borg-
wraitii
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Commits
rP23915: Delete paired wall technologies.
Summary

Currently players have a choice between a pair of wall technologies:

  • “Rubble Materials” gives city walls −20% build time, −1 crush armor.
  • “Geometric Masonry” gives city walls +10% build time, +2 crush armor.

They're available to all civilizations, however:

  • They can't be researched at Iberian walls (presumably because they start with them), though Iberians can research the technology at wall towers captured from other civilizations, and the Iberian starting walls do benefit from them.
  • They can be researched at the Roman siege wall tower, though siege walls are unaffected by them.

This seems wrong.
Now the Britons and Gauls have walls that visibly made from rubble, whereas Carthage etc. have walls that visibly consist of geometric masonry, therefore this choice breaks immersion.
Moreover, a build time reduction is mostly useful prior to building them, but they're researched at finished wall towers, so the choice is obvious (a difference of three crush armour means it takes rams effectively 37% longer to destroy them).

In short, I'm questioning the entire concept and I believe it's better to simply delete these technologies, hence this patch.

Test Plan

Agree this is an improvement.

Diff Detail

Repository
rP 0 A.D. Public Repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Nescio created this revision.Thu, Jul 16, 3:27 PM
Owners added a subscriber: Restricted Owners Package.Thu, Jul 16, 3:27 PM

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

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

borg- added a subscriber: borg-.EditedThu, Jul 16, 6:10 PM

Yes, i agree with delete this techs of walls.

These technologies can be added and used in the Civic center. A possibility to reduce construction time and another to reduce costs, just an idea.

borg- accepted this revision.Mon, Jul 27, 5:42 PM
This revision is now accepted and ready to land.Mon, Jul 27, 5:42 PM
wraitii updated this revision to Diff 12983.Sat, Aug 1, 10:20 AM

Rebased.

wraitii accepted this revision.Sat, Aug 1, 10:21 AM

I agree that these techs make no sense and removing them seems good.

This works. I'll commit once the lights are green.

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

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

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

Linter detected issues:

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

This revision was landed with ongoing or failed builds.Sat, Aug 1, 12:34 PM
This revision was automatically updated to reflect the committed changes.