[5.1.3] Large amount of CPU usage and Memory

Diometus commented 1 month ago

WoW Version

WoW Retail (all expansions up to The War Within)

Issue

I am using Addon Usage to monitor why my PC was being bogged down after adding BLU and found that it is eating up my memory (25% Memory at 54,337.7k) and my CPU (47% CPU at 5.97ms/s and rising). There must be something going on like a memory leak.

PC

I use an i5 -13600K with a 4070 Ti Super and 32 GB DDR5. The game is on a Samsung 990 Pro m.2 (7500MB/s).

List of Addons that are getting loaded

Note: that the 'Version:' is taken from the '## Version:' within their toc file_

Addon Usage - Version: 3.2.2
AddonFactory by Thaoky - Version: 11.0.004
Adventure Guide Lockouts - Version: 1.5.1
Altoholic by Thaoky - Version: 11.0.004
Ask Mr. Robot - Version: 146
Auctionator - Version: 11.0.11
Azeroth Pilot Reloaded - Version: v4.5.0
Bazooka - Version: v3.1.2
Better Fishing - Version: 1.1.27
Better Level-Up! - v5.1.3  - Version: v5.1.3
BtWQuests - Version: 2.48.1
ChatCopyPaste - Version: 1.22
DataStore by Thaoky - Version: 2024.09.15
DataStore_Achievements - Version: 2024.09.15
DataStore_Agenda - Version: 2024.09.15
DataStore_Auctions - Version: 2024.09.15
DataStore_Characters - Version: 2024.09.15
DataStore_Containers - Version: 2024.09.15
DataStore_Crafts - Version: 2024.09.15
DataStore_Currencies - Version: 2024.09.15
DataStore_Garrisons - Version: 2024.09.15
DataStore_Inventory - Version: 2024.09.15
DataStore_Mails - Version: 2024.09.15
DataStore_Pets - Version: 2024.09.15
DataStore_Quests - Version: 2024.09.15
DataStore_Reputations - Version: 2024.09.15
DataStore_Spells - Version: 2024.09.15
DataStore_Stats - Version: 2024.09.15
DataStore_Talents - Version: 2024.09.15
Details! Cast Log (plugin) - Version: N/A
Details! Damage Meter - Version: #Details.13072.161
Details!: Advanced Death Logs (plugin) - Version: N/A
Details!: Compare 2.0 - Version: N/A
Details!: Elitism (plugin) - Version: v2.15.1
Details!: Encounter Breakdown (plugin) - Version: N/A
Details!: Raid Check (plugin) - Version: N/A
Details!: Tiny Threat (plugin) - Version: N/A
Details!: Vanguard (plugin) - Version: N/A
ElitismHelper - Version: 0.14.0
FrameColor - Version: 3.7.5
FrameColor_Skin_WeakAurasOptions - Version: 1.0.2
GatherMate2 - Version: 1.50.2.1
GatherMate2 <Gas Clouds> - Version: 2.6
GSE - Version: 3.2.14-e
GSE Data Feeds and Minimap - Version: 3.2.14-e
GSE Utility Functions - Version: 3.2.14-e
GSE: izzi Evoker Macros - Version: v1
HandyNotes - Version: v1.6.21
HandyNotes: Battle for Azeroth - Version: 101
HandyNotes: Cataclysm - Version: 101
HandyNotes: Dragonflight - Version: 101
HandyNotes: Legion - Version: 101
HandyNotes: MapNotes - Version: 2.3.9
HandyNotes: Mechagon & Nazjatar - Version: 17
HandyNotes: Mists of Pandaria - Version: 101
HandyNotes: Shadowlands - Version: 101
HandyNotes: The Burning Crusade - Version: 101
HandyNotes: The War Within - Version: 101
HandyNotes: TravelGuide - Version: v2.19.0
HandyNotes: Visions Of N'Zoth - Version: 42
HandyNotes: Warlords of Draenor - Version: 101
HandyNotes: World of Warcraft - Version: 101
HandyNotes: Wrath of the Lich King - Version: 101
HandyNotes_WorldMapButton - Version: 110002.02-Release
MaxDps - Version: 11.0.36
MoveAny by D4KiR   - Version: 1.7.24
Narcissus - Version: 1.7.0
Outfitter - Version: 11.0.0.6
Routes - Version: v1.8.3
Routes Import/Export - Version: v1.0.10
RoutesGatherMate2IntegrationFix - Version: 1.0.0
RoutesMiniMap - Version: v1.0.0
WeakAuras - Version: 5.17.1
Wowhead Quick Link - Version: 2.16.3

SoulCookie commented 1 month ago

[+1]

I had to disable BLU because it was tanking my FPS into single-digits-- not just bad, but unplayable.

donniedice commented 1 month ago

Workin on this now. I may push a version with pet stuff disabled for now while I work it out.

I noticed a bit of lag but thought it was my potato.

donniedice commented 1 month ago

v5.1.4 // v5.1.5 have the removed battlepets.lua. Battle Pet level tracking logic isn't working properly.

Going to keep this issue open until I resolve the Pet Battle Logic and the associated memory leak.

donniedice commented 1 month ago

New release fixes issues with memory and cpu usage.

v5.1.5

Removed - Debug for missing localization - [localizaton.lua]
Removed - Excess debug output - [core.lua, initialization.lua, utils.lua]
Removed - (temp) Redacted - 'Battle-Pets Level-Up!' - [core.lua, initialization.lua, , options.lua, utils.lua]
Removed - Unused Dependancy - [BLU.toc, BLU_Vanilla.toc, BLU_Cata.toc]

https://github.com/donniedice/BLU/releases/tag/v5.1.5

I'll fully re-add Battle-Pets soon tm.

donniedice / BLU