Exploration on bot tick lag-spike (lag causing found) about camod HOT 7 OPEN

@Inq8
@darkademic

The scanning of HarvesterBotModuleCA is the causing of heavy lag-spikes.

After I set "ScanForIdleHarvestersInterval" to 800, the lag-spikes almost disappear. Although AI also not building harvesters correctly at this point.

If you also confirm that, I guess you have to check the HarvesterBotModuleCA which seems to have very serious performance issue, especially in the late game at certain map.

from camod.

@darkademic

The new Harvester Module still cause lag spikes here but more unpredictable (so I am not certain if it is really causing the lag spikes)

from camod.

The new Harvester Module still cause lag spikes here but more unpredictable (so I am not certain if it is really causing the lag spikes)

Thanks for the info.

It's pretty strange as I've played multiple hundreds of bot games using the trait (in its old form, and a few in updated form) and not noticed lag spikes like that.

I haven't tried on the map you mentioned though, so can investigate more.

The changes to HarvesterBotModule are pretty small, the AI just counts how many harvesters it has, and compares it to the MaxHarvesters config, to determine whether to produce more, whereas in the normal version it just compares to the number of refineries, so it'd be odd if that caused a major change in performance, though I'm not particularly well versed in C# or game programming.

from camod.

In fact, me neither on those game program.

I happened to find this map, because at that time, I thought protection squad caused the spike lags so a map could trigger protection actions frequently can be good (by minefield at middle and harvester get frequently attacked) . It was very confusing when I found the protection squad was not the cause.

from camod.

It is said found actors in world is a high cost measure, but it is useful and vital to any bot module. We can only use as less as we can.

from camod.

The lagspike is expected in the harvester bot module on specific maps. It shouldn't be CA-specific because it shouldn't come from the scanning but from the pathfinder though which is called during assigning idle harvesters to work.

Nonetheless, I have a minor performance optimization suggestion here:

which I would rewrite to

var numHarvesters = AIUtils.CountActorByCommonName(Info.HarvesterTypes, player);

if (numHarvesters >= Info.MaxHarvesters)
	return;

var harvCountTooLow = numHarvesters < AIUtils.CountBuildingByCommonName(Info.RefineryTypes, player) * Info.HarvestersPerRefinery;
if (!harvCountTooLow )
	return;

var harvInfo = AIUtils.GetInfoByCommonName(Info.HarvesterTypes, player);
if (unitBuilder.RequestedProductionCount(bot, harvInfo.Name) == 0)
	unitBuilder.RequestUnitProduction(bot, harvInfo.Name);

to avoid the iteration on the harvester info on the GetInfoByCommonName call if the end result isn't even used.

from camod.

Therefore, as you can see I drop this AI module on SP AI. I mean the original one.

from camod.