Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#4412 closed enhancement (fixed)

[PATCH] Minimum distance for pillars

Reported by: elexis Owned by: elexis
Priority: Nice to Have Milestone: Alpha 22
Component: UI & Simulation Keywords: patch
Cc: Patch:

Description

Besides keeping territory, the pillars are only useful for their aura. In particular the mauryan one giving a (non-stackable) 75m trade speed bonus aura, it is hard to estimate where to place the next pillar to cover the area seamlessly. Also placing many monuments too close to each other inflates them, reduces their uniqueness.

The attached patch adds a minDistance restriction to give the player a clear feedback where building monuments is useful. (2x aura range for maximum trade router coverage, 1x aura range for the iberian monument to still allow overlapping)

Attachments (1)

pillar_minDistance.diff (2.4 KB) - added by elexis 2 years ago.

Download all attachments as: .zip

Change History (11)

Changed 2 years ago by elexis

Attachment: pillar_minDistance.diff added

comment:1 Changed 2 years ago by stanislas69

Shouldn't that be covered by some kind of terrain overlay, just like the one used in Delenda Est ?

Last edited 2 years ago by stanislas69 (previous) (diff)

comment:2 Changed 2 years ago by elexis

Screenshot and/or link to the related code?

comment:3 Changed 2 years ago by bb

Isn't it better to implement for fixing #3915 for more then only attack ranges? This approach seems a bit hacky to me, why should it be disabled to build multiple pillars close, as it is useless anyway? (except for territory maybe)

comment:4 Changed 2 years ago by elexis

Indeed that visualization would be preferable. At least a small build restriction still seems preferable to me for esthetical reasons, but the ticket can be closed as won't fix too.

comment:5 Changed 2 years ago by mimo

I think that patch is still useful, at least for the iber monument. The idea is not to be able to cover all your territory with this aura, but to place it at some strategical position. So such a limitation make some sense by itself, although i would have put a value bigger than the range to avoid such a full coverage, maybe twice the range? For the pillar, i'm not sure it is useful. But if kept, it makes sense to have a full coverage of the trader path, and thus the limitation should be smaller than twice the radius.

comment:6 Changed 2 years ago by elexis

The 20% pillar profit seems significant, as it can qualify the production of 20% more units. It can be buffed, but I'm cautious as trade income already has a big impact in the late game. (It was added in #3639 since an attack bonus didn't seem to fit and the historical pillar was built in popular places, spreading the culture, trade and stuff.)

comment:8 Changed 2 years ago by elexis

Keywords: rfc removed

comment:9 Changed 2 years ago by elexis

Owner: set to elexis
Resolution: fixed
Status: newclosed

In 19348:

Pillar / Monument cleanup.

Prevent Iberian players from covering their entire territory with the attack bonus giving monument as proposed by mimo.
Indirect aura range indication with the build restrictions for the trade bonus yielding mauryan pillar.
Disable capturing of mauryan pillar equally to the iberian one following rP18024 (so that the entire territory doesn't switch ownership quickly).
Add the missing name for the mauryan pillar, so that the aura bonus tooltip shows up, as reported by Grugnas.

Differential Revision: https://code.wildfiregames.com/D196
Reviewed By: Grugnas
Fixes #4412

comment:10 Changed 2 years ago by elexis

Milestone: Work In ProgressAlpha 22
Note: See TracTickets for help on using tickets.