What's been happening with the update texts for each new build? It's become increasingly difficult to find what is changing between each alpha, to decide if it's worth grabbing it to test out.
(eg, old builds showed each change and to which files; new builds only seem to show the latest beta/release summary, maybe occasionally adding something to that list)
It would indeed be nice to be able to see the commit log in addition to the custom changelog I use, but unfortunately that doesn't seem to be possible (I add the commit notes just as before, but they aren't visible).
The reason I use a custom changelog is that it's a lot easier to release betas with a concise changelog without having to manually update it for each file every time I release a beta or a new version. The automatically generated changelog sucks for releases as the changes are stretched over several pages, and also contains notes that aren't crucial to include.
The custom changelog on the other hand is a bit harsh to update every time I commit so there's the problem you're mentioning. The commit notes are posted on IRC at #wowace every time there's an update, perhaps that would alleviate problem. Alternatively you can see them with an SVN client. I'll look into providing alphas with a slightly better changelog :)
Hello again, I managed to fix PW:Shield too! It was showing for more than 1k lower than what the blizzard tooltip states(which is identical to combat log report so it is correct).
There was still a slight difference after that, so I checked Twin Disciplines and it was marked as a bonus to "all", now I don't know if all covers shields too but switching the code of it to this
did the trick for me as it's the average hit is now only 2 absorption points below the real one. I have to mention that when I tried to put the "PW:S" part after "all" it didn't work but in front of it made it work.
I have to mention that when I tried to put the "PW:S" part after "all" it didn't work but in front of it made it work.
because it's incorrect syntax. adding the comma and the "All" reference isn't being applied to the list of spells, it's a new field for the table with no reference to any drd key.
Hello again, I managed to fix PW:Shield too! It was showing for more than 1k lower than what the blizzard tooltip states(which is identical to combat log report so it is correct).
There was still a slight difference after that, so I checked Twin Disciplines and it was marked as a bonus to "all", now I don't know if all covers shields too but switching the code of it to this
did the trick for me as it's the average hit is now only 2 absorption points below the real one. I have to mention that when I tried to put the "PW:S" part after "all" it didn't work but in front of it made it work.
Thanks for the testing! I've added these changes, except the 2nd change correctly, which would be Spells = { "All", "Power Word: Shield" }
The reason "All" doesn't apply to Power Word: Shield is that it's a special case with it disabled due to the spell not being a standard heal.
Sorry, I'm new here. Not sure how this type of question is recieved. Has there been any interest in updating this addon? I took a hiatus from WoW and feel absolutely gimped without it. It's worth a kickstarter to get this thing running avian. Dr. Damage will be greatly missed!
Has there been any interest in updating this addon?
It was never even fully updated for Cataclysm, and the last update of any kind was over a year ago. At this point it would need to be pretty much rewritten from the ground up, and it's unlikely anyone will bother.
Fortunately, this kind of addon doesn't really have any purpose nowadays, since the in-game spell tooltips show the actual amount of damage or healing your spells do, rather than the old tooltips that only showed the base amount without accounting for your attack power, spell damage, talents, etc. If you still want to see the amounts on your action buttons, it wouldn't be that hard to write such an addon, but it wouldn't have anything to do with DrDamage either; it would just be a simple "scan the tooltip, pull out the number" affair. You might even find someone willing to write it for you if you post in the addon search forum, and probably better luck posting on WoWInterface since the WowAce forums here are largely dead these days.
I'll look into it.
It would indeed be nice to be able to see the commit log in addition to the custom changelog I use, but unfortunately that doesn't seem to be possible (I add the commit notes just as before, but they aren't visible).
The reason I use a custom changelog is that it's a lot easier to release betas with a concise changelog without having to manually update it for each file every time I release a beta or a new version. The automatically generated changelog sucks for releases as the changes are stretched over several pages, and also contains notes that aren't crucial to include.
The custom changelog on the other hand is a bit harsh to update every time I commit so there's the problem you're mentioning. The commit notes are posted on IRC at #wowace every time there's an update, perhaps that would alleviate problem. Alternatively you can see them with an SVN client. I'll look into providing alphas with a slightly better changelog :)
Added.
According to one of the dev's comments over on Curse (http://wow.curse.com/downloads/wow-addons/details/dr-damage.aspx#831715):
Initial Cataclysm release available.
Just a heads up for you, Grace for priests is broken. The code is
While it should be
First of all while the talent Imp.Power Word: Shield states it's 5/10% it's actually 10/20%(refer to http://elitistjerks.com/f77/t105330-cataclysm_healing_priest_theorycrafting/p4/#post1784551) so I changed the code for this talent to that
There was still a slight difference after that, so I checked Twin Disciplines and it was marked as a bonus to "all", now I don't know if all covers shields too but switching the code of it to this
did the trick for me as it's the average hit is now only 2 absorption points below the real one. I have to mention that when I tried to put the "PW:S" part after "all" it didn't work but in front of it made it work.
because it's incorrect syntax. adding the comma and the "All" reference isn't being applied to the list of spells, it's a new field for the table with no reference to any drd key.
Thanks for the testing! I've added these changes, except the 2nd change correctly, which would be Spells = { "All", "Power Word: Shield" }
The reason "All" doesn't apply to Power Word: Shield is that it's a special case with it disabled due to the spell not being a standard heal.
Fixed.
It was never even fully updated for Cataclysm, and the last update of any kind was over a year ago. At this point it would need to be pretty much rewritten from the ground up, and it's unlikely anyone will bother.
Fortunately, this kind of addon doesn't really have any purpose nowadays, since the in-game spell tooltips show the actual amount of damage or healing your spells do, rather than the old tooltips that only showed the base amount without accounting for your attack power, spell damage, talents, etc. If you still want to see the amounts on your action buttons, it wouldn't be that hard to write such an addon, but it wouldn't have anything to do with DrDamage either; it would just be a simple "scan the tooltip, pull out the number" affair. You might even find someone willing to write it for you if you post in the addon search forum, and probably better luck posting on WoWInterface since the WowAce forums here are largely dead these days.
http://www.curse.com/addons/wow/spelldamage