• 0

    posted a message on DrDamage - Official Thread
    Quote from Headhuntress
    Paladin Holy Radiance shouldn't have "NoDotHaste = true ". It does in fact gain ticks through haste.


    Fixed.
    Posted in: General AddOns
  • 0

    posted a message on DrDamage - Official Thread
    Quote from Headhuntress
    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).

    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
    --Improved Power Word: Shield (multiplicative - 3.3.3, updated for 4.0.3)
            [GetSpellInfo(14748)] = {     [1] = { Effect = 0.1, Spells = "Power Word: Shield", ModType = "Improved Power Word: Shield" }, },
    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
    --Twin Disciplines (additive - 3.3.3, updated for 4.0.3)
            [GetSpellInfo(47586)] = {     [1] = { Effect = 0.02, Spells = "Power Word: Shield","All"  }, },
    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.
    Posted in: General AddOns
  • 0

    posted a message on DrDamage - Official Thread
    Quote from Radicalb21
    Any chance this mod will be updated for WoW 4.0.1?


    Initial Cataclysm release available.
    Posted in: General AddOns
  • 0

    posted a message on DrDamage - Official Thread
    Quote from sr1030nx
    Could you add the demon abilites of Metamorphosis?

    Shadow Cleave http://www.wowhead.com/?spell=50581
    Immolation http://www.wowhead.com/?spell=50589

    Immolation scales with spell pwr.
    Im not sure about Shadow Cleave, but it is like maul, where it pops on your next melee strike.


    Added.
    Posted in: General AddOns
  • 0

    posted a message on DrDamage - Official Thread
    Quote from Nich
    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 :)
    Posted in: General AddOns
  • 0

    posted a message on DrDamage - Official Thread
    Quote from sr1030nx
    Could you add the demon abilites of Metamorphosis?

    Shadow Cleave http://www.wowhead.com/?spell=50581
    Immolation http://www.wowhead.com/?spell=50589

    Immolation scales with spell pwr.
    Im not sure about Shadow Cleave, but it is like maul, where it pops on your next melee strike.


    I'll look into it.
    Posted in: General AddOns
  • 0

    posted a message on DrDamage - Official Thread
    Quote from Harpz
    Getting:
    1x DrDamage-1.7.2_release\Melee.lua:454: Usage: GetItemInfo(itemID|"name"|"itemlink")
    DrDamage-1.7.2_release\Melee.lua:454: in function `Melee_InventoryChanged'
    DrDamage-1.7.2_release\DrDamage.lua:949: in function `?'
    CallbackHandler-1.0-5 (Ace3):147: in function <...Ons\Ace3\CallbackHandler-1.0\CallbackHandler-1.0.lua:147>
    <string>:"safecall Dispatcher[2]":4: in function <[string "safecall Dispatcher[2]"]:4>
    <in C code>: ?
    <string>:"safecall Dispatcher[2]":13: in function `?'
    CallbackHandler-1.0-5 (Ace3):92: in function `Fire'
    AceEvent-3.0-3 (Ace3):120: in function <Ace3\AceEvent-3.0\AceEvent-3.0.lua:119>
    
      ---


    This code hasn't been touched for several versions and I've never seen or heard about this error happening before. This shouldn't technically even be happening but I've added an additional check to make sure it doesn't :)

    Out of interest, did you get it more than once?

    Download the latest alpha r1166 or newer for the fix:

    http://www.wowace.com/addons/dr-damage/files/
    Posted in: General AddOns
  • 0

    posted a message on LibHealComm-4.0
    Regarding discussion about flash of light, holy light and holy shock coefficients check:

    http://www.wowace.com/addons/dr-damage/tickets/122-holy-paladin-spellpower-coefficients/#comments
    Posted in: Libraries
  • 0

    posted a message on DrDamage - Official Thread
    Quote from lilsparky

    i don't know how the modification stuff really works, but it seems to be modifying the current values which would be kind of a backwards way to get where i'd like to go. it would require "undoing" the current talents and then redoing the new ones rather than just recalculating base stats from scratch.


    Not that your idea doesn't have merit, but simply adding a "Set = true" key in the modification table allows you to set all base values instead of modifying the existing values, which I could make the default just as well if that's easier, since not anything else uses the code :)

    *EDIT*

    Let me elaborate a bit, so for example the idea is:

    DrDamage.CasterGlobalModify = {
       ["Fire"] = { ["spellDmg"] = 1080, ["critPerc"] = 30.5, Set = true },
       ["Shadow"] = { ["spellDmg"] = 1300, ["critPerc"] = 35, Set = true },
    }
    This would set the fire school base spell damage to 1080 and crit chance to 30.5%. And shadow school base spell damage to 1300 and crit chance to 35%.


    The function also supports overriding abilities, the priority is spell school < spell type < spell name. So anything related to the specific spell will override the other setting if you use the Set = true attribute. Without the Set = true attribute the override function will add to the base value.

    DrDamage.CasterGlobalModify = {
       ["Fire"] = { ["spellDmg"] = 1080, ["critPerc"] = 30.5, Set = true },
       ["Destruction"] = { ["spellDmg"] = 1300, Set = true },
       ["Conflagrate"] = { ["spellDmg"] = 1500, ["critPerc"] = 40, ["castTime"] = 2.3, Set = true },
     }
    - This would set the fire school to base 1080 spelldamage and 30.5% crit
    - Destruction spells would override the default spell damage and use 1300 spelldamage, even if they're fire school (but they'd still use 30.5% crit if they're fire destruction spells)
    - Conflagrate would use 1500 base spell damage and a 40% crit and a 2.3 second cast time (just as an example).


    DrDamage.CasterGlobalModify = {
       ["Fire"] = { ["spellDmg"] = 1080, Set = true },
       ["Conflagrate"] = { ["spellDmg"] = 100 },
     }
    - This would set the fire school to 1080 spell damage and Conflagrate would use 1180 spell damage.


    After this base modification talents handled by DrDamage are calculated on top of this normally, which you can feed into DrDamage (I can improve support of that if needed)

    *EDIT2*

    This would be easier if you'd get on IRC for example, then we can custom tailor whatever you need.
    Posted in: General AddOns
  • 0

    posted a message on DrDamage - Official Thread
    Quote from lilsparky
    i really love drdamage, but i'm dying to for a means of "trying on" different talent specs. i had written a mod that connect talented templates to drd a while back, but drd only obeyed certain talents (those that aren't accounted for in the generic base stats) so the project was kind of pointless.

    i know you've stated you have no desire to mimic the base stat calcs, but would you be open to a slight reconfiguring of the system to allow a separate mod to plug into the core to achieve that? what i'm thinking is that you could make the api calls a bit more hookable. realizing i'm not totally familiar with the drd code, what i'm thinking is you could have something like:

    drd:GetBaseStat(stat, spellid)
    spellid would be the current spellid in question (and might not even be considered by the function)
    stat would be whatever base stat is interesting from "str" to "spellpower" to "armor"... not that they'd all be interesting to drd...

    drd's internal code for that function would simply be a table lookup based on the stat token. since drd deals with specific spell talents later in the pipeline, it would retun the true base stat.

    what this would allow is another mod to come in and plug into drd very simply by intercepting that function to provide recalculated base stats from proposed talent specs (using talented or maybe even the blizz api at this point since you can set talents and then accept them when done). i'm thinking the effort to recalculate would probably not be too great given that you have a simple means of verification -- the actual values from the api.


    it would be so cool to be able to use a talent calculator and see what the net effect is on my different spells given my current gear. take a point from this and put it in that and now what do i get?


    Your idea is interesting and I think I've stated before that anyone wanting to make a plugin for this kind of purposes has my support. As I currently don't any longer have that much time to work on the addon I would want to make the needed changes as simple as possible. Actually, I've already added support for outside calculation modification quite a while ago, but since nobody has expressed actual interest for writing a plugin to use this functionality, it's currently commented out and I think it's only available in the caster module. It's not exactly what you proposed but it could work for your purposes.

    The caster module is setup to receive a global modification table for setting base values or modifying base values of calculation variables. If you look at the current Caster.lua lines 718-752. This was implemented just as a proof of concept and I'm sure it can be improved, so please give suggestions. Also I'm not actually certain it meets your needs as I haven't looked into that much.

    Please tell me what you think and I hope we can make this work :)
    Posted in: General AddOns
  • 0

    posted a message on DrDamage - Official Thread
    Quote from HunterZ
    Nice! Will give it a spin in a couple of hours.


    I'll still add the LDB config launcher later today.

    *EDIT*

    Added in 1.7.1_beta-2.
    Posted in: General AddOns
  • 0

    posted a message on DrDamage - Official Thread
    Newest beta has been ported over to Ace3.
    Posted in: General AddOns
  • 0

    posted a message on DrDamage - Official Thread
    Quote from HunterZ
    The numbers came back on their own at some point, not sure what was going on.


    Well I think the issue is due to patches clearing game cache.. and after a cache clear DrDamage is unable to retrieve any weapon info on startup (as it takes up to 15 seconds for the server to provide it). Re-equipping the weapon or relogging will fix the issue, and it's only apparent after your cache has been deleted.

    But to prevent this in the future, in case the original weapon scan was unable to retrieve info, I've set DrDamage to check it again after 10 seconds. That should solve the problem in most cases unless the server is heavily lagged, but I also don't want to set the delay very long as then the user is without any info for that period (during a load without any cache, if cache is available it works normally).
    Posted in: General AddOns
  • 0

    posted a message on DrDamage - Official Thread
    Quote from HunterZ
    Using r1076 on WoW 3.2.2 enUS, my action bar numbers are all showing zero for my Hunter in either MM or SV spec. Didn't have time to check on any other toons.

    Also: armor penetration calculations have changed across the board in 3.2.2 (not sure what the new numbers are), and Hunters' Volley coefficient has been increased from 0.0586 to 0.0837.

    Edit: ArP info: http://blue.mmo-champion.com/28/19717353372-armor-pen-nerf-in-322.html



    On my test hunter numbers were visible as they should, so I'm not exactly sure what the issue is about. Have you tried re-equipping your weapon?
    Posted in: General AddOns
  • 0

    posted a message on DrDamage - Official Thread
    Quote from Ydrisselle
    A Broker plugin (launcher I think) + LibQTip/LibQTipClick clickable tooltip? :)


    I guess LibQTip would warrant some testing then.. now I only need to figure out how to do that without a subscription..
    Posted in: General AddOns
  • To post a comment, please or register a new account.