You need an ordered set by insertion, all the others options are just doing the same with extra steps. Maybe you can skip the set and do a linked list if you can guarantee no insert repetition.

Requires an update and rebase.

This is technically not looking for the resource nearest to the dropsite.

An alternative that I would maybe prefer would be to generalise the 'military colony' building, with an 'outpost' building that's relatively cheaper & weaker, but acts as a territory root to claim territory. Possibly we'd want to give another bonus to civil centres though.

Civic centre costs ought to be revisited regardless of this patch. Right now they're very expensive, making it really hard to rebuild after your last centre has been destroyed. I think e.g. 600 wood, 300 stone, 0 metal would be more reasonable. Anyway, that's something for a different patch.

Certainly the patch encourages the use of new ccs and military camps and makes the use of theaters more interesting. It also opens up a possibility for new auras / technologies to gain territory.

Good patch. I will include only quinquireme.

Thank you! I've tried your patch, it works, and it's really much better than the current situation!
I'd even go as far as saying that if there is an A24c, this should be included.

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

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

Update AI.

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

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

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

Restore to the previous version + rebase

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

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

More extensive testing.

In D1202#157175, @Stan wrote:

One thing you can do is close this one (abandon) and commandeer the other one so we can finish it.

I didn't commit it cause I could not reproduce the bug D1593 looks simple enough for me to commit it. One thing you can do is close this one (abandon) and commandeer the other one so we can finish it.

I'm not sure which code is correct either. Mine is taken from official documentation but looks dirtier

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

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

Change merc-tech.

Happens :)

In D3303#157162, @Freagarach wrote:

Why is that?
[EDIT] Ah, I think you mean ResourceGatherer/BaseSpeed?

Yeah, I did: the current situation (replacing the rates of each resource sub-type) was done to ensure treasures were still gatherable; now they're in a separate node, that technology file can be cleaned up significantly.
When writing my previous reply I copied and pasted the wrong lines, I apologize for the confusion, I should have checked before clicking “Submit”.

In D3303#157158, @Nescio wrote:

You should also replace the last modifications in simulation/data/technologies/unit_mercenary.json with:

		{ "value": "TreasureCollecter/MaxDistance", "replace": 0 }

or something.

Why is that?

In D3303#157158, @Nescio wrote:

(What happened with the Rate?)

Not yet implemented, so no change with how it worked before.

I really wish we would have gone for alphabetical schema ordering, it's soooo easy to autofix. I did it for hyrule and it's pretty cool.

Apparently I misunderstood, thank you for the clarifications.
I fully agree this patch should just contain the necessary code to support resource regeneration, without actually changing gameplay here, which can be done later in mods or future patches.

You should also replace the last modifications in simulation/data/technologies/unit_mercenary.json with:

		{ "value": "TreasureCollecter/MaxDistance", "replace": 0 }

or something. (What happened with the Rate?)

@Nescio, this diff does not change template values, that is for another diff. It _can_ be done after this one.