Petra does not use Chasing string in her codebase -> ok.

@Feldfeld whats your opinion ?

Dunno if we need it in vanilla. It rewards players to not use their units directly?

Only citizen soldiers are able to promote so having them garrisoned for most time will hurt economy of the player. And experience gain is not so fast as in combat.

[16:18] <Freagarach> One could perhaps check the timer every time the rate is computed?
[16:19] <Stan`> could be delete this.trickleTimer
[16:20] <Freagarach> That too, and var -> let ;)

(http://irclogs.wildfiregames.com/2020-03/2020-03-21-QuakeNet-%230ad-dev.log)

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

@Feldfeld any objections ?

You can also request changes :D

If it is historically accurate I have nothing against it. It could help kushite a little bit.

Didn't mean to accept btw

I agree that the build time needs to be reduced, however not sure if it should be to 100. Carthaginian embassies have 150 build time, too

I agree with the change as well, and I think this will be an improvement.

_ CC, Storehouse, Farmhouse, House, Farm, Barrack, Minitower, Vision Tower, Mini CC (if any), Market, Tower, Temple - Armery (i don't know how to order them),..... Stable, Darsena (i try to order them by priority and useness.)

the former will stop in its tracks

@Stan _ CC, Storehouse, Farmhouse, House, Farm, Barrack, Minitower, Vision Tower, Mini CC (if any), Market, Tower, Temple - Armery (i don't know how to order them),..... Stable, Darsena (i try to order them by priority and useness.)

Need to fix tests.

Should we disable the button when it cannot be used?

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

• Remove warns.
• Fix linter issue.

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

Fix test, thanks @Stan!

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.

Added corral aura.

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.

Thanks for the hard work already done @Stan :D

Some different approach.
Test fails for now for some unknown reason. Perhaps @elexis can shed their light on that? (Hence the warnings still in place.)

@Stockfish0ad can you elaborate on which one you'd like to change?

Hello, yes i think CC, should be colocated in first place, but, following that order, then some more buildings maybe should be changed too.
Greetings

Separately, could you create a donkey trader for Athens and Sparta? Southern Greece is rather mountaineous, horses were scarce, and donkeys (and mules!) were used extensively.
Macedon and Seleucids can keep the horse-and-cart trader.

I can probably center on the compound.

No, I don't think it makes sense that ships packed with champions are faster than empty ships. Or that larger ships (more garrison space) benefit significantly more.

Is it an either/or situation? Or can you centre on the centre of the donkey + human compound? Or the donkey's head and front legs?

The patch was not working properly. I just recopied the files. I'm still not sure why the last attempt produced garbage.

See #5695

Might be line endings. Depending on your OS, it might or might not strip the special chars

This patch still isn't working correctly. Please standby; I'm having to figure out why svn diff is producing something unusable. Anybody have any thoughts?

Why don't you generate the patch in the root https://code.wildfiregames.com/source/0ad/browse/ps/trunk/

instead of https://code.wildfiregames.com/source/0ad/browse/ps/trunk/source/tools/lobbybots/

In D2630#111755, @user1 wrote:

Per elexis' suggestion, this patch doesn't rename or move things anymore.

It should happen that a follow-up revision to this will restructure the directory tree and rename everything as they need to be.

Tests still pass.

Per elexis' suggestion, this patch doesn't rename or move things anymore.

In my opinion that cav should be slower than basic cavs, as @Stan said, if we want to make a realistic game that cav should be slower. But is a fact that if we change that, cavalry heavy would be even less used, so maybe we should buff it a bit.

Clone is done in rP23529.

@Nescio is it still something we might want? If so can you commandeer and add it to the list of your patches?

In D2640#110676, @Nescio wrote:

Notice that donkey traders are centred on the human, not the donkey (@Stan).

I guess you can abandon this. We generally avoid this kind of patches as it makes SVN blame harder.

History checks out, so removing the obelisk makes sense. Thanks for the patch.

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

Fix test.

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.

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

• Rebased.
• Some inlines.

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

fix linter comments

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

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