diff --git a/src/PerformanceMonitor.cpp b/src/PerformanceMonitor.cpp index 6219576e..dfe2fbd5 100644 --- a/src/PerformanceMonitor.cpp +++ b/src/PerformanceMonitor.cpp @@ -78,13 +78,13 @@ void PerformanceMonitor::PrintStats(bool perTick, bool fullStack) switch (i->first) { case PERF_MON_TRIGGER: - key = "T"; + key = "Trigger"; break; case PERF_MON_VALUE: - key = "V"; + key = "Value"; break; case PERF_MON_ACTION: - key = "A"; + key = "Action"; break; case PERF_MON_RNDBOT: key = "RndBot"; @@ -152,8 +152,8 @@ void PerformanceMonitor::PrintStats(bool perTick, bool fullStack) } else { - float fullTickCount = data[PERF_MON_TOTAL]["RandomPlayerbotMgr::FullTick"]->count; - float fullTickTotalTime = data[PERF_MON_TOTAL]["RandomPlayerbotMgr::FullTick"]->totalTime; + float fullTickCount = data[PERF_MON_TOTAL]["PlayerbotAIBase::FullTick"]->count; + float fullTickTotalTime = data[PERF_MON_TOTAL]["PlayerbotAIBase::FullTick"]->totalTime; LOG_INFO( "playerbots", @@ -173,13 +173,13 @@ void PerformanceMonitor::PrintStats(bool perTick, bool fullStack) switch (i->first) { case PERF_MON_TRIGGER: - key = "T"; + key = "Trigger"; break; case PERF_MON_VALUE: - key = "V"; + key = "Value"; break; case PERF_MON_ACTION: - key = "A"; + key = "Action"; break; case PERF_MON_RNDBOT: key = "RndBot"; @@ -231,7 +231,7 @@ void PerformanceMonitor::PrintStats(bool perTick, bool fullStack) time, minTime, maxTime, avg, amount, key.c_str(), disName.c_str()); } } - if (PERF_MON_TOTAL != i->first) + if (i->first != PERF_MON_TOTAL) { float tPerc = (float)typeTotalTime / (float)fullTickTotalTime * 100.0f; float tTime = (float)typeTotalTime / fullTickCount / 1000.0f; @@ -278,7 +278,7 @@ void PerformanceMonitorOperation::finish() std::chrono::microseconds finished = (std::chrono::time_point_cast(std::chrono::high_resolution_clock::now())) .time_since_epoch(); - uint64_t elapsed = (finished - started).count(); + uint64 elapsed = (finished - started).count(); std::lock_guard guard(data->lock); if (elapsed > 0) diff --git a/src/PlayerbotAI.cpp b/src/PlayerbotAI.cpp index b9d14573..7a18c520 100644 --- a/src/PlayerbotAI.cpp +++ b/src/PlayerbotAI.cpp @@ -4758,40 +4758,50 @@ Item* PlayerbotAI::FindStoneFor(Item* weapon) const return stone; } -static const std::vector uPriorizedWizardOilIds = -{ - MINOR_WIZARD_OIL, LESSER_WIZARD_OIL, BRILLIANT_WIZARD_OIL, WIZARD_OIL, SUPERIOR_WIZARD_OIL -}; - -static const std::vector uPriorizedManaOilIds = -{ - MINOR_MANA_OIL, LESSER_MANA_OIL, BRILLIANT_MANA_OIL, SUPERIOR_MANA_OIL, -}; - Item* PlayerbotAI::FindOilFor(Item* weapon) const { + if (!weapon) + return nullptr; + + const ItemTemplate* item_template = weapon->GetTemplate(); + if (!item_template) + return nullptr; + + // static const will only get created once whatever the call amout + static const std::vector uPriorizedWizardOilIds = + { + MINOR_WIZARD_OIL, MINOR_MANA_OIL, + LESSER_WIZARD_OIL, LESSER_MANA_OIL, + BRILLIANT_WIZARD_OIL, BRILLIANT_MANA_OIL, + WIZARD_OIL, SUPERIOR_MANA_OIL, SUPERIOR_WIZARD_OIL + }; + + // static const will only get created once whatever the call amout + static const std::vector uPriorizedManaOilIds = + { + MINOR_MANA_OIL, MINOR_WIZARD_OIL, + LESSER_MANA_OIL, LESSER_WIZARD_OIL, + BRILLIANT_MANA_OIL, BRILLIANT_WIZARD_OIL, + SUPERIOR_MANA_OIL, WIZARD_OIL, SUPERIOR_WIZARD_OIL + }; + Item* oil = nullptr; - ItemTemplate const* pProto = weapon->GetTemplate(); - if (pProto && (pProto->SubClass == ITEM_SUBCLASS_WEAPON_SWORD || pProto->SubClass == ITEM_SUBCLASS_WEAPON_STAFF || - pProto->SubClass == ITEM_SUBCLASS_WEAPON_DAGGER)) + if (item_template->SubClass == ITEM_SUBCLASS_WEAPON_SWORD || + item_template->SubClass == ITEM_SUBCLASS_WEAPON_STAFF || + item_template->SubClass == ITEM_SUBCLASS_WEAPON_DAGGER) { for (const auto& id : uPriorizedWizardOilIds) { - oil = FindConsumable(uPriorizedWizardOilIds[id]); - if (!oil) - oil = FindConsumable(uPriorizedManaOilIds[id]); - if (oil) + if (oil = FindConsumable(id)) return oil; } } - else if (pProto && (pProto->SubClass == ITEM_SUBCLASS_WEAPON_MACE || pProto->SubClass == ITEM_SUBCLASS_WEAPON_MACE2)) + else if (item_template->SubClass == ITEM_SUBCLASS_WEAPON_MACE || + item_template->SubClass == ITEM_SUBCLASS_WEAPON_MACE2) { for (const auto& id : uPriorizedManaOilIds) { - oil = FindConsumable(uPriorizedManaOilIds[id]); - if (!oil) - oil = FindConsumable(uPriorizedWizardOilIds[id]); - if (oil) + if (oil = FindConsumable(id)) return oil; } } @@ -5493,6 +5503,7 @@ uint8 PlayerbotAI::FindEquipSlot(ItemTemplate const* proto, uint32 slot, bool sw // no free position return NULL_SLOT; } + bool PlayerbotAI::IsSafe(Player* player) { return player && player->GetMapId() == bot->GetMapId() && player->GetInstanceId() == bot->GetInstanceId() && !player->IsBeingTeleported(); @@ -5676,3 +5687,4 @@ std::set PlayerbotAI::GetCurrentIncompleteQuestIds() return result; } + diff --git a/src/PlayerbotAIBase.cpp b/src/PlayerbotAIBase.cpp index 57b40727..b26a3e64 100644 --- a/src/PlayerbotAIBase.cpp +++ b/src/PlayerbotAIBase.cpp @@ -11,6 +11,11 @@ PlayerbotAIBase::PlayerbotAIBase(bool isBotAI) : nextAICheckDelay(0), _isBotAI(i void PlayerbotAIBase::UpdateAI(uint32 elapsed, bool minimal) { + if (totalPmo) + totalPmo->finish(); + + totalPmo = sPerformanceMonitor->start(PERF_MON_TOTAL, "PlayerbotAIBase::FullTick"); + if (nextAICheckDelay > elapsed) nextAICheckDelay -= elapsed; else diff --git a/src/PlayerbotAIBase.h b/src/PlayerbotAIBase.h index 66fec542..a1322211 100644 --- a/src/PlayerbotAIBase.h +++ b/src/PlayerbotAIBase.h @@ -24,6 +24,7 @@ public: protected: uint32 nextAICheckDelay; + class PerformanceMonitorOperation* totalPmo = nullptr; private: bool _isBotAI; diff --git a/src/RandomPlayerbotMgr.cpp b/src/RandomPlayerbotMgr.cpp index 29bc672d..cc806663 100644 --- a/src/RandomPlayerbotMgr.cpp +++ b/src/RandomPlayerbotMgr.cpp @@ -153,7 +153,7 @@ double botPIDImpl::calculate(double setpoint, double pv) botPIDImpl::~botPIDImpl() {} -RandomPlayerbotMgr::RandomPlayerbotMgr() : PlayerbotHolder(), processTicks(0), totalPmo(nullptr) +RandomPlayerbotMgr::RandomPlayerbotMgr() : PlayerbotHolder(), processTicks(0) { playersLevel = sPlayerbotAIConfig->randombotStartingLevel; diff --git a/src/RandomPlayerbotMgr.h b/src/RandomPlayerbotMgr.h index cfc82dfb..f1e80ab6 100644 --- a/src/RandomPlayerbotMgr.h +++ b/src/RandomPlayerbotMgr.h @@ -204,9 +204,8 @@ private: std::list currentBots; uint32 bgBotsCount; uint32 playersLevel; - PerformanceMonitorOperation* totalPmo; }; #define sRandomPlayerbotMgr RandomPlayerbotMgr::instance() -#endif \ No newline at end of file +#endif diff --git a/src/cs_playerbots.cpp b/src/cs_playerbots.cpp index a96dc6e3..e39b7f33 100644 --- a/src/cs_playerbots.cpp +++ b/src/cs_playerbots.cpp @@ -83,6 +83,16 @@ public: return true; } + if (!strcmp(args, "toggle")) + { + sPlayerbotAIConfig->perfMonEnabled = !sPlayerbotAIConfig->perfMonEnabled; + if (sPlayerbotAIConfig->perfMonEnabled) + LOG_INFO("playerbots", "Performance monitor enabled"); + else + LOG_INFO("playerbots", "Performance monitor disabled"); + return true; + } + sPerformanceMonitor->PrintStats(); return true; }