Page MenuHomeWildfire Games

[gameplay] disable pers hall and ishtar gate

Authored by Nescio on May 17 2020, 2:31 PM.


Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
rP23888: Disable Persian Hall & Ishtar gate as buildable structures.

This patch makes the Persian Hall and Ishtar Gate unbuildable.

0 A.D.'s interpretation of the Kardakes is rather problematic; see this forum thread.

Only after his defeat by Alexander the Great at the Battle of the Issus (333 BC), Darius III sent for a new army. At the Battle of Gaugamela (331 BC), which marked the end of the Achaemenid Empire, about 15 Indian elephants (Arrian Anabasis 3.8.6) participated. This is the one and only known case for the Persian use of war elephants. Apparently they're a novelty. War elephants are not mentioned by Herodotus (covering history up to c. 480 BC), nor by Xenophon (describing the situation c. 400 BC).

The hall actor itself is loosely based on the Gate of All Nations in Persepolis. However, 0 A.D.'s version has entrances only on two sides; the most important one, one the south, facing Darius's Apadana, is missing.

The famous Ishtar Gate of Babylon is Neo-Babylonian, i.e. predating the Achaemenids. Allowing pers to build it is anachronistic, akin to ptol building the Great Pyramid at Giza, or brit building Stonehenge.

Taking everything into account, it's probably best to disable these historically inaccurate assets for now.

Test Plan

Check for mistakes and omissions.

Diff Detail

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

Event Timeline

Nescio created this revision.May 17 2020, 2:31 PM
Owners added a subscriber: Restricted Owners Package.May 17 2020, 2:32 PM
Nescio added inline comments.May 17 2020, 2:34 PM
26 ↗(On Diff #11900)

iber monument, kush pyramids, maur pillar, etc. are not CC either.

54–57 ↗(On Diff #11900)

identical to parent

31 ↗(On Diff #11900)

Aura has its own tooltip.

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

Link to build:

I agree for the most part. I've always hated the "Persian Hall" as a buildable structure.

As far as war elephants go for the Persians, perhaps keep them and give them a low training limit of 5.

borg- added a subscriber: borg-.Jun 9 2020, 3:26 PM

if it is historically inaccurate, then I think I must agree.
Are kardakes inaccurate too, or can they be moved to another building? In fact hoplites are "useless" (immortals is the same), but javelins would be interesting to keep.

Nescio added a comment.Jun 9 2020, 7:05 PM

Are kardakes inaccurate too,

Yes, they are, that's actually what triggered this patch, see forum discussion (link in summary).

borg- accepted this revision.Jul 13 2020, 6:15 PM
This revision is now accepted and ready to land.Jul 13 2020, 6:15 PM
wraitii accepted this revision.Jul 23 2020, 11:18 AM
wraitii added a subscriber: wraitii.

Fine for me, will commit tomorrow.

I'm taking @borg-'s word that this doesn't unbalance the Persians.

There seems to be a valid argument for giving Persians "mercenary" elephants. They were used at the Battle of Gaugamela, (alongside Chariots which we reintroduced ;) ). Further, the Aechemenid empire, following Darius 1's conquests, did have access to India and its elephants.

I think we should perhaps reintroduce them, possibly in a mercenary camp of sorts. Likewise, giving access to hellenised hoplites might be interesting.

That being said, Persia already has a ridiculously large roster :p .

That being said, I agree with the arguments made above, so this will be for later.

wraitii updated this revision to Diff 12902.Jul 25 2020, 9:44 AM

Rebased for committing.

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

Link to build:

This revision was landed with ongoing or failed builds.Jul 25 2020, 9:54 AM
This revision was automatically updated to reflect the committed changes.
Nescio added inline comments.Jul 25 2020, 11:04 AM

This tooltip is meaningful (see line 49) and should have been kept (cf. rome_arch.xml).
There might be more mistakes in other recent commits, so I'll include restoring this line in a future fix oversights patch.

wraitii added inline comments.Jul 25 2020, 11:05 AM

Ah, thanks. I think your original diff removed the tooltip, and even though it changes I just removed it.


Nescio added inline comments.Jul 25 2020, 11:10 AM

Yes, the old superfluous tooltip was replaced only a few days ago by D2892/rP23865.