games/vcmi
1
0
Fork 0
mirror of https://github.com/vcmi/vcmi.git synced 2026-06-19 22:57:37 +02:00
Code Issues Releases Activity

Merge pull request #6405 from IvanSavenko/ai_log

Browse Source 
Reduce AI logging to avoid performance hit
This commit is contained in:
Ivan Savenko
2025-12-08 20:42:15 +02:00
committed by GitHub
parent 0f0f08f8fe 8ecf1f52e9
commit e8f341eeaa
2 changed files with 15 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files
AI/Nullkiller2/Analyzers/DangerHitMapAnalyzer.cpp
+2 -1
View File
@@ -154,10 +154,11 @@ void DangerHitMapAnalyzer::updateHitMap()
// TODO: Mircea: Why is this danger calculated so differently than FuzzyHelper::evaluateDanger?
// shouldn't it use the path danger instead of hero strength?
newThreat.danger = path.getHeroStrength();
#if NK2AI_TRACE_LEVEL >= 1
auto danger2 = aiNk->dangerEvaluator->evaluateDanger(pos, path.targetHero);
logAi->trace("Danger comparison for (%d %d %d) %s is %f vs %f",
pos.x, pos.y, pos.z, path.targetHero->getNameTranslated(), newThreat.danger, danger2);
#endif
if(newThreat.value() > node.maximumDanger.value())
{
node.maximumDanger = newThreat;
AI/Nullkiller2/Engine/PriorityEvaluator.cpp
+13 -12
View File
@@ -1512,37 +1512,37 @@ float PriorityEvaluator::evaluate(Goals::TSubgoal task, int priorityTier)
{
if (evaluationContext.enemyHeroDangerRatio > dangerThreshold && !evaluationContext.isDefend && priorityTier != PriorityTier::FAR_HUNTER_GATHER)
{
logAi->trace("case PriorityTier::FAR_HUNTER_GATHER if 1");
// logAi->trace("case PriorityTier::FAR_HUNTER_GATHER if 1");
return 0;
}
if (evaluationContext.buildingCost.marketValue() > 0)
{
logAi->trace("case PriorityTier::FAR_HUNTER_GATHER if 2");
// logAi->trace("case PriorityTier::FAR_HUNTER_GATHER if 2");
return 0;
}
if (priorityTier != PriorityTier::FAR_HUNTER_GATHER && evaluationContext.isDefend && (evaluationContext.enemyHeroDangerRatio > dangerThreshold || evaluationContext.threatTurns > 0 || evaluationContext.turn > 0))
{
logAi->trace("case PriorityTier::FAR_HUNTER_GATHER if 3");
// logAi->trace("case PriorityTier::FAR_HUNTER_GATHER if 3");
return 0;
}
if (evaluationContext.explorePriority == 3)
{
logAi->trace("case PriorityTier::FAR_HUNTER_GATHER if 4");
// logAi->trace("case PriorityTier::FAR_HUNTER_GATHER if 4");
return 0;
}
if (priorityTier != PriorityTier::FAR_HUNTER_GATHER && ((evaluationContext.enemyHeroDangerRatio > 0 && arriveNextWeek) || evaluationContext.enemyHeroDangerRatio > dangerThreshold))
{
logAi->trace("case PriorityTier::FAR_HUNTER_GATHER if 5");
// logAi->trace("case PriorityTier::FAR_HUNTER_GATHER if 5");
return 0;
}
if (maxWillingToLose - evaluationContext.armyLossRatio < 0)
{
logAi->trace("case PriorityTier::FAR_HUNTER_GATHER if 6");
// logAi->trace("case PriorityTier::FAR_HUNTER_GATHER if 6");
return 0;
}
if (vstd::isAlmostZero(evaluationContext.armyLossRatio) && evaluationContext.closestWayRatio < 1.0)
{
logAi->trace("case PriorityTier::FAR_HUNTER_GATHER if 7");
// logAi->trace("case PriorityTier::FAR_HUNTER_GATHER if 7");
return 0;
}
@@ -1554,34 +1554,35 @@ float PriorityEvaluator::evaluate(Goals::TSubgoal task, int priorityTier)
score -= evaluationContext.goldCost / 2; // don't include the full cost of School of Magic or others because those locations are beneficial
score -= evaluationContext.armyInvolvement * evaluationContext.armyLossRatio * 0.1;
#if NK2AI_TRACE_LEVEL >= 1
logAi->trace("case PriorityTier::FAR_HUNTER_GATHER score %f, strategicalValue %f, goldReward %f, skillRewardMultiplied %f, armyReward %f, armyGrowth %f, goldCost -%f, armyInvolvementMultiplied -%f, "
"armyLossPersentage %f, movementCost %f, enemyHeroDangerRatio %f",
score, evaluationContext.strategicalValue, evaluationContext.goldReward, evaluationContext.skillReward * evaluationContext.armyInvolvement * (1 - evaluationContext.armyLossRatio) * 0.05,
evaluationContext.armyReward, evaluationContext.armyGrowth, evaluationContext.goldCost, evaluationContext.armyInvolvement * evaluationContext.armyLossRatio,
evaluationContext.armyLossRatio, evaluationContext.movementCost, evaluationContext.enemyHeroDangerRatio);
#endif
if (score > 0)
{
// score = 1000;
if (evaluationContext.movementCost > 0)
{
logAi->trace("case PriorityTier::FAR_HUNTER_GATHER if 8");
// logAi->trace("case PriorityTier::FAR_HUNTER_GATHER if 8");
score -= evaluationContext.movementCost / 20 * score; // we expect movement won't be over 20 turns
}
// TODO: Mircea: This doesn't make sense at it always seems to be 0. To test
if(evaluationContext.enemyHeroDangerRatio > 0)
{
logAi->trace("case PriorityTier::FAR_HUNTER_GATHER if 9");
// logAi->trace("case PriorityTier::FAR_HUNTER_GATHER if 9");
score *= 1 - evaluationContext.enemyHeroDangerRatio;
}
if(evaluationContext.armyLossRatio > 0)
{
logAi->trace("case PriorityTier::FAR_HUNTER_GATHER if 10");
// logAi->trace("case PriorityTier::FAR_HUNTER_GATHER if 10");
score *= 1 - evaluationContext.armyLossRatio;
}
}
logAi->trace("case PriorityTier::FAR_HUNTER_GATHER score final %f", score);
// logAi->trace("case PriorityTier::FAR_HUNTER_GATHER score final %f", score);
break;
}
case PriorityTier::LOW_PRIO_EXPLORE: