gptbot/0ad
1
0
Fork 0
forked from mirrors/0ad
Code Pull Requests Activity

Workaround to reduce the impact of AI performance problems when planning new bases.

Browse Source 
When no free territory was left on the map, the AI tried to place a CC
every 10th AI turn.
That's about all 10 seconds and it freezed the game for about a second.
Now when you played against 4 players it freezed aobut 4 seconds every
10 seconds, which is very bad.
I wanted to solve this properly but realized that I'd have to change a
lot in the terrain analysis code that will probably be replaced anyway
when we have a new pathfinder with a proper interface.
Also removes an unused variable (creating a territory map and not using
it can't be good for performance either).

This was SVN commit r14859.
This commit is contained in:
Yves
2014-03-20 14:26:15 +00:00
parent 6eaad6cbb6
commit 2eeb7c2ba1
1 changed files with 13 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
binaries/data/mods/public/simulation/ai/aegis/headquarters.js
+13 -4
View File
@@ -530,8 +530,6 @@ m.HQ.prototype.findBestEcoCCLocation = function(gameState, resource){
// Then checks for a good spot in the territory. If none, and town/city phase, checks outside
// The AI will currently not build a CC if it wouldn't connect with an existing CC.
var territory = m.createTerritoryMap(gameState);
var obstructions = m.createObstructionMap(gameState, 0);
obstructions.expandInfluences();
@@ -869,9 +867,20 @@ m.HQ.prototype.checkBasesRessLevel = function(gameState,queues) {
|| capacity[type] < gameState.getOwnUnits().filter(API3.Filters.and(API3.Filters.byMetadata(PlayerID, "subrole", "gatherer"), API3.Filters.byMetadata(PlayerID, "gather-type", type))).length * 1.05)
{
// plan a new base.
if (gameState.countFoundationsByType(gameState.applyCiv("structures/{civ}_civil_centre"), true) === 0 && queues.civilCentre.length() === 0) {
if (this.outOf[type] && gameState.ai.playedTurn % 10 !== 0)
if (gameState.countFoundationsByType(gameState.applyCiv("structures/{civ}_civil_centre"), true) === 0 && queues.civilCentre.length() === 0) {
// In endgame when the whole map is claimed by players, we won't find a spot for a new CC.
// findBestEcoCCLocation needs to search the whole map for a good spot and is currently way too slow, so we wait quite long
// until we check again. The "PlayerID * 10" part is to distribute the load across multiple turns.
// TODO: this is a workaround. the current solution is bad for various reasons:
// 1. findBestEcoCCLocation could be much more efficient for the case when nearly all territory is occupied.
// 2. It doesn't make sense to check the whole map for a good spot for all resource types if we could see in the beginning
// that no free territory is available to build a CC.
// 3. Trying to build a new CC should not only be triggered by the need for more resources.
// Opportunity (having destroyed an enemy CC and some soldiers standing around) is also a good reason for a new CC.
// 4. Last but not least it causes the AI to react slowly when new territory becomes available.
if (this.outOf[type] && gameState.ai.playedTurn % 100 !== PlayerID * 10)
continue;
var pos = this.findBestEcoCCLocation(gameState, type);
if (!pos)
{