DrDamage - Official Thread

DrDamage displays the calculated, average damage or healing of abilities with talents, gear and buffs included on your actionbar buttons. The addon will also add various information to the default tooltips in your spellbook and on the actionbar.

Fantastic to see that someone has taken this up. My first request would be for macro support in much the same way as TheoryCraft did whereby it would take the name of the macro as the name of the spell. Would be great to have this because thanks to FastCast being broken, all my spells are now macro'd with /stopcasting before them. It would be great if it could do one of two things when reading the spellname from the macro name. If the macro was named "Fireball" it would use the highest rank in the tooltip/action button calculations. If it was named "Fireball(8)" then it would use that specific rank. It would also be good if there was some limited abbreviation support because of the very short macro name length.

Fantastic to see that someone has taken this up. My first request would be for macro support in much the same way as TheoryCraft did whereby it would take the name of the macro as the name of the spell. Would be great to have this because thanks to FastCast being broken, all my spells are now macro'd with /stopcasting before them. It would be great if it could do one of two things when reading the spellname from the macro name. If the macro was named "Fireball" it would use the highest rank in the tooltip/action button calculations. If it was named "Fireball(8)" then it would use that specific rank. It would also be good if there was some limited abbreviation support because of the very short macro name length.

In WoW 2.03 i get the following error when attempting to mouseover a spell,

DrDamage.lua:1187 Usage: GetSpellCritChanceFromIntellect("unit")

Perhaps its just due to changes in an api. This happens on Test Realm as well as Beta Realms which are both currently WoW 2.03.

Also thank you for the pyroblast update, there is still issues with how talents are calculated to average damage.

Pyroblasts +damage calculations are now correct however the way talents are factored are still off.
Currently it takes the base rank 8 values 708, 898 then adds (spelldamage*1.27) then takes that total *1.13 for the talents fire power (10%) and playing with fire (3%). However fire power is applied to the base damage of pyroblast and all fire spells, pre +damage. 778, 988 the new base values + (spelldamage*1.27)*1.03 gives the correct damage. With those talents. So is there a way to change the MageTable to have the addon read the tooltip instead of use the base values you have in the SpellData.lua since the default tooltip is updated when that the fire power talent is applied.

I believe this mechanic is also true for arcane instability and piercing ice.

In WoW 2.03 i get the following error when attempting to mouseover a spell,

DrDamage.lua:1187 Usage: GetSpellCritChanceFromIntellect("unit")

Perhaps its just due to changes in an api. This happens on Test Realm as well as Beta Realms which are both currently WoW 2.03.

Also thank you for the pyroblast update, there is still issues with how talents are calculated to average damage.

Pyroblasts +damage calculations are now correct however the way talents are factored are still off.
Currently it takes the base rank 8 values 708, 898 then adds (spelldamage*1.27) then takes that total *1.13 for the talents fire power (10%) and playing with fire (3%). However fire power is applied to the base damage of pyroblast and all fire spells, pre +damage. 778, 988 the new base values + (spelldamage*1.27)*1.03 gives the correct damage. With those talents. So is there a way to change the MageTable to have the addon read the tooltip instead of use the base values you have in the SpellData.lua since the default tooltip is updated when that the fire power talent is applied.

I believe this mechanic is also true for arcane instability and piercing ice.

It's a change in the API, but I can put it in however. Passing it to the current one won't cause an error but is unnecessary.

Alright, I assumed fire power works after damage gear. It's no problem to set it up to only increase base damage, if it actually works that way, but I'm pretty sure it doesn't. I had a guy run tests with fireball and fire power does scale after gear.

The reasons I don't use tooltips for reading spell's base damage is:

1. Needs a lot of localization
2. Some spells get talents into the tooltips, while other's dont (would need to make a table on which do and which dont)
3. Hard to test and write without having access to all classes

The amount has been limited to max one per 1.5 seconds. I plan to make options to disable this (particularily target scanning) if someone's concerned. Also looking into how to make the recalculation a bit more intelligent, i.e. calculating only if the changes really affect you.

I did some extensive testing with a custom fire build of only fire power talent in regards to changing spell damage. It seems Fireball does indeed receive 10% increase to base and +damage however pyroblast does not, pyroblast receives the increase to the base damage only. This may be a bug or perhaps intentional. Im sorry for the confusion, i will be doing more testing later today on other fire talents. So based on the current version fireball is correct but pyroblast is not. I have reported this issue in beta and live realm bug forum.

another issue with Fire Power, the spell Fire Blast is receiving nearly a 14% increase in base damage instead of 10%, this makes the tooltip off a bit. This is a bug, so im not sure its worth changing the mod.

Are you officially working on Druid support or should I just patch that in myself?

Well I haven't started yet but I plan to whenever I get the time for it.

Quote from Spock »

On my shadow priest I noticed that Vampiric Touch (41pt shadow talent) doesnt get any calculations done, either in tooltip or on button.

Other then that it all seems to work perfact, great work! :)

I've added VT to the next version.

Quote from delusionary »

another issue with Fire Power, the spell Fire Blast is receiving nearly a 14% increase in base damage instead of 10%, this makes the tooltip off a bit. This is a bug, so im not sure its worth changing the mod.

I think that's actually not because of Fire Power but the fact that spells receive a small base damage bonus for each level-up after you've received the spell until you get the next rank. Fire Blast rank 7 is learned at level 54. Thus it has received a bonus perhaps equal to around 4%. I'm trying to come up with a formula to account for this effect, but it doesn't seem that easy. Fireball is correct because it is learned at level 60. On warlock and shaman tooltips it's not a problem cause I've added the base damage ranges that correspond to a level 60 character, but mages have their base damage ranges set according to thottbot.