CPU usage too high
tflo opened this issue · 8 comments
You said:
For performance for ABP, I need to run through and optimize some things. It has a high mem and high CPU reported by the "Addon Usage" addon when my character is in a big city. ABP goes garbage collection just fine though.
…and I said:
The first time I used your addon (a few months ago) I was also shocked by this. But then I learned that addons that are first in the alphabet tend to get blamed for memory and CPU usage of the AceLibs they contain, which should actually be distributed to all addons.
In case you haven't, download and install the Ace Library separately and activate it in your addon manager. This should save some addons from being falsely accused, ABP being one of them:
[…]
The samples are a bit bad, because they were taken without much action, spellcasting, moving etc. But the general picture is OK. ABP goes a bit higher when really doing some fighting or such, but not that much.
Unfortunately it seems you were right and I was wrong, especially since one of your recent updates.
After installing 2023.6.1, it happened that I was in a raid group for the Researchers Under Fire event, and I got near-freezes, i.e FPS at 1 or below during combat. The FPS returned to normal immediately after combat.
I then repeated the event on another toon and with CPU profiling enabled, got similar FPS issues, and the CPU report is this:
Of course, I cannot say with certainty that ABP is the responsible for the near-freezes, but this is completely unexpected and I have never seen it like this before. Normally, with a lot of combat, Hekili is the leader with about 40% CPU (it does a lot of calculations during combat, so I would consider this normal), followed at about 5%–15% by WeakAuras, TSM (WTF is it doing in combat?!!), OPie, M6, ….
ABP used to be in the 5%–15% group or below that group.
I then rolled back to the ABP version before 2023.6.1, repeated the thing, and ABP was not that high as before but still higher than what I would consider appropriate. I don’t have a screenshot but it was the leader of the 5%–15% group or above.
So it seems that with or without version 2023.6.1, but especially with version 2023.6.1, ABP’s CPU usage – during combat – is indeed far too high. Even more so when you consider that I have no combat-related spells on an ABP bar. This is the full screenshot from before, in the raid group with the ABP bars highlighted:
Only toys and items at that point.
After that, I went back to using Button Forge in place of ABP, and this thing has absolutely low CPU usage. Perhaps you can get an idea of what makes the difference by looking at their code. (I haven't done this, so this is merely hypothetical).
– Tom
To get a more accurate CPU Usage, Ace3 needs to be excluded in ABP (with Lib Ace3 addon installed) in _ExtLib.xml.
Hi @tflo,
Thank you for bringing up the CPU usage issue. I wanted to let you know that, at the moment, it's best to avoid using ABP/ABP-M6 for quite a while. I'm currently working on some significant improvements, but these changes may or may not meet the expectations of power users like yourself.
Cheers,
- Tony
at the moment, it's best to avoid using ABP/ABP-M6 for quite a while.
Yeah, I came more or less to the same conclusion, hence my "Back to Button Forge" ;)
But thanks for the honest statement and confirmation.
Oh, and btw…
power users like yourself
…in terms of game content, I'm not a power user at all! In the last two years, I have joined maybe one M+ and one normal raid. Heroic dungeons and open world raid groups (like in the example) are probably the most combat and CPU intensive stuff I do. I'm just too old for it ;-)
Hi Tom,
That way I've approach the creation of ABP is mediocre at best and this is probably why perf issues are showing. I've been so busy with other stuff that I am contemplating doing a stop dev on ABP.
Hi Tom,
That way I've approach the creation of ABP is mediocre at best and this is probably why perf issues are showing. I've been so busy with other stuff that I am contemplating doing a stop dev on ABP.
Sorry to hear that, or maybe it's good, if the "other stuff" is important. In any case, ABP will remain on my "stay tuned for updates' list. Thanks again!