Elitist Jerks
Register
Blogs
Forums


Go Back   Elitist Jerks » Paladins

Reply
 
LinkBack Thread Tools
Old 04/29/09, 2:16 PM   #76
Thaeryn
Piston Honda
 
Thaeryn's Avatar
 
Tauren Druid
 
Dethecus
I'm pretty up in the air about the range check thing... I feel like I'm already sliding too far into the "hand-holding" area with this addon. To be fair, if you are focused on using the addon the whole fight, you'll probably be dead to the 101 types of fire or AoE damage in the game. Then again, the more info on my screen, the better. I don't want a "this is how to play your class perfectly" addon, but I do want to keep a high amount of utility.

I'll consider the range check, but no promises. It also depends on how much work it would be... I'm notoriously lazy.

As far as the sub-menu... I'm a bit stumped as well. The paladin I originally wrote this for is having the same problem, and she said it was showing up under SexyMap as well (GREAT addon btw). Does it show up under any other addons for anyone else, or could this be something specific to SexyMap?

Offline
Reply With Quote
Old 04/29/09, 2:38 PM   #77
Heck
Von Kaiser
 
Heck's Avatar
 
Blood Elf Paladin
 
Velen
Yeah, it's under SexyMap for me as well. Not a huge issue for me as long as I know where it is. If you can't figure it out, I"d add something to the readme file to let people know where to find it if it's missing.

Offline
Reply With Quote
Old 04/29/09, 7:09 PM   #78
rea123
Glass Joe
 
Human Warrior
 
Azjol-Nerub (EU)
In point of fact the reason I've jumped on this mod is that I found myself more aware of my surroundings while using it, since I wasn't staring at action bars (a while back) or cooldown bars (more recently) working out what was going to fall next and what key it was on. The "current" icon is large enough to be easily recognised in peripheral vision, and the fact that the priority is packed into one (main) icon means that it requires less concentration. Or that's what I found, anyway.

So it turns out that with 5 level 80s, 3 of them dual-spec'd, I'm now participating in 8 threads, each dominated by how underpowered they are. Boy did I make some bad decisions.

Offline
Reply With Quote
Old 04/30/09, 12:05 PM   #79
 frmorrison
Protector
 
frmorrison's Avatar
 
Ashstorm
Human Paladin
 
No WoW Account
Originally Posted by Thaeryn View Post
I'll consider the range check, but no promises. It also depends on how much work it would be... I'm notoriously lazy.
This mod does something pretty well, helps you to know which buttons to press when in melee range. I think that is enough, I wouldn't want to see this mod get feature creep.

I have had issues with Sexymap putting stuff underneath it before.

Millions of words are written annually purporting to tell how to beat the races, whereas the best possible advice on the subject is found in the three monosyllables: 'Do not try.'

United States Offline
Reply With Quote
Old 04/30/09, 4:20 PM   #80
Ahhnold
Glass Joe
 
Gnome Warlock
 
Cairne
Thaeryn: Kudos on such a useful addon. I am also against having a range checker.

The only suggestions that I think would make the mod even more useful would be to have multiple priority profiles (mainly just for fighting undead) and maybe an option to make the Avenging Wrath icon flash when it is off of cooldown to make it more visible. I still sometimes forget about it during battle.

Thank you.

Offline
Reply With Quote
Old 05/01/09, 11:57 AM   #81
Covertghost
Von Kaiser
 
Blood Elf Paladin
 
Turalyon
Great mod, a toggled cooldown graphic might be pretty useful though (for judging how close things are to clashing, you could really get rid of your bars entirely if this was done)

Offline
Reply With Quote
Old 05/01/09, 11:09 PM   #82
rea123
Glass Joe
 
Human Warrior
 
Azjol-Nerub (EU)
Hey,

OK - as stated above, I am strongly against range being taken into account for the spell decisions, and I think the majority of posters so far are in agreement. However, like I say, I do like the idea of the mod displaying some indication of out-of-rangeness for fights where finding the edge of the hit-box is preferable. As it happens, I've enjoyed playing with this mod, and am in the process of writing a Fury equivalent for my main character's off-spec, as I am entirely new to the wonderful world of warrior DPS. While doing so, I implemented the sort of range checking that I have in mind, so it was natural for me to come back and hack this into my own version of S.H.I.T.

Now I'm not going to ask for these changes to be incorporated into the official version, but I guess it can't hurt to share my tweaks in case others are like-minded.

However, I must make it clear right now, that while I may be able to help with questions regarding these changes, you will be monkeying around with the addon code. If you follow these steps and later have an issue with the mod, I do not want to hear of Thaeryn being hassled with support queries. Warranty is void if seal is broken! Also, naturally, the changes are version-specific - v1094. No promises on future updates.

Anyway, the changes I made were as follows. In the method:

function SHIT:OnUpdate(elapsed)
I modified the final "else" to become "end", and removed the following "end". So:

...
    elseif SHITdb.showon == 3 and GetSpellInfo(SHIT.L["Crusader Strike"]) ~= null then
        if UnitClassification("target") ~= "worldboss" then
            return
        end
    end

    SHIT:DecideSpells()
 
end
Then following the line:

	SHIT.textureList["current"]:SetTexture(GetSpellTexture(spell))
I added the following code:

      SHIT.textureList["current"]:SetVertexColor(1, 1, 1);

      local range = IsSpellInRange(spell, "target");
      if range ~= nil and range == 0 then
            SHIT.textureList["current"]:SetVertexColor(0.8, 0.1, 0.1);
      end

So it turns out that with 5 level 80s, 3 of them dual-spec'd, I'm now participating in 8 threads, each dominated by how underpowered they are. Boy did I make some bad decisions.

Offline
Reply With Quote
Old 05/02/09, 7:37 PM   #83
Logicality
Glass Joe
 
Draenei Paladin
 
Misha
I am going to be trying this out soon, just noticed something strange regarding Avenging Wrath in your wording. You state that the add-on prioritizes it for when there is a free GCD, but it is off the GCD, so it really should just be prioritized when it is up.

Offline
Reply With Quote
Old 05/02/09, 7:58 PM   #84
Kallell
Von Kaiser
 
Blood Elf Paladin
 
Mug'thol
Not sure if this is just my computer or not, but the mod seems to reset the transparency everytime I log in. I set it to 0, and next time I log in, the slider is still at 0, but it has a normal level of transparency again. If I move it up and then back down, it properly disappears. Thanks for a great mod!

Offline
Reply With Quote
Old 05/02/09, 7:58 PM   #85
Tiberius
Glass Joe
 
Human Paladin
 
Azgalor
Originally Posted by Thaeryn View Post
I'm pretty up in the air about the range check thing... I feel like I'm already sliding too far into the "hand-holding" area with this addon. To be fair, if you are focused on using the addon the whole fight, you'll probably be dead to the 101 types of fire or AoE damage in the game. Then again, the more info on my screen, the better. I don't want a "this is how to play your class perfectly" addon, but I do want to keep a high amount of utility.

I'll consider the range check, but no promises. It also depends on how much work it would be... I'm notoriously lazy.

As far as the sub-menu... I'm a bit stumped as well. The paladin I originally wrote this for is having the same problem, and she said it was showing up under SexyMap as well (GREAT addon btw). Does it show up under any other addons for anyone else, or could this be something specific to SexyMap?
For the SexyMap problem could you put the priority tab in a minimize (+) under SHIT so it isn't on the main interface menu with the other addons unless you maximize it and it show the priority tab.

Offline
Reply With Quote
Old 05/06/09, 9:58 AM   #86
valygiarth
Glass Joe
 
Human Paladin
 
Proudmoore (EU)
I'd like to thank the author & testers.
This mod is really awesome.

Offline
Reply With Quote
Old 05/14/09, 6:59 PM   #87
Heck
Von Kaiser
 
Heck's Avatar
 
Blood Elf Paladin
 
Velen
So this is now officially my favorite Paladin-specific mod... ahead of Pally Power. I can't remember the last time I've looked at my toolbar, other than to swap which judgement I'm using.

One tweak I'd like to see is to add a cooldown counter to the "on deck" ability. In some situations where my "Use Now" ability is on a short CD (usually less than 2 seconds) I find myself automatically looking at what's the next ability "on deck" (the indicator to the right of center). I've been trying to ween myself away from my instinctive "spam judgement" habit, and I always find myself looking for crusader strike to pop up. If the on deck ability had a counter, I could make a snap decision if I'd rather wait for the next ability (which may be higher up in our FCFS combat system) or hit the one that is coming off cooldown first. This is really only an issue when the cooldowns of the two abilities are within the span of a single GCD.

The configuration menu still seems to be showing up under SexyMap for some reason. I know where it hides, though, so not a huge issue.

Offline
Reply With Quote
Old 05/15/09, 5:31 AM   #88
rea123
Glass Joe
 
Human Warrior
 
Azjol-Nerub (EU)
I doubt that would be a hard change to implement, but I'm not sure it would be a good idea for the purposes you state. FCFS does not have a priority system; FCFS means First Come First Serve, and means literally that. FCFS with clash resolution is where the priority system comes into play, and for optimal DPS, clash resolution should only be invoked for two skills with identical cooldowns; specifically ones whose cooldown will be 0 when the current GCD expires.

In this case, the mod correctly shows the higher priority. If the "next" ability is higher in your priority list, that will be because it will come off cooldown after the "current" one does, so choosing next over current would be a violation of FCFS.

Of course a skilled player may have situational reasons to break from FCFS - low mana or AoE groups that the tool is not entirely suited for - but I would be concerned that implementing a feature to disregard FCFS lies outside the scope of this tool.

So it turns out that with 5 level 80s, 3 of them dual-spec'd, I'm now participating in 8 threads, each dominated by how underpowered they are. Boy did I make some bad decisions.

Offline
Reply With Quote
Old 05/15/09, 9:42 AM   #89
Exemplar
Bald Bull
 
Human Paladin
 
Scarlet Crusade
Originally Posted by Heck View Post
If the on deck ability had a counter, I could make a snap decision if I'd rather wait for the next ability (which may be higher up in our FCFS combat system) or hit the one that is coming off cooldown first. This is really only an issue when the cooldowns of the two abilities are within the span of a single GCD.
Why? As proven over in the Ret thread (with math), delaying a currently available ability (even our "worst" damage ability) by 0.1 seconds for a "better" ability (including our "best") to come off of cooldown is a loss of DPS. Always hit what is currently available - never wait.

Rock: "We're sub-standard DPS. Nerf Paper, Scissors are fine."
Paper: "OMG, WTF, Scissors!"
Scissors: "Rock is OP and Paper are QQers. We need PvP buffs."

Offline
Reply With Quote
Old 05/15/09, 9:46 AM   #90
beatific
fishsticks.
 
beatific's Avatar
 
Tauren Druid
 
Kel'Thuzad
Originally Posted by Heck View Post
Used this last night during our continuation on Mimiron. Invaluable tool. Let me keep an eye on my character instead of looking at bars waiting for abilities to come off cooldown.

I did notice that it would occasionally "go off", but like has been said above, I assumed that it just meant everything was on CD.

Being able to add in AoW or Sacred Shield as the lowest priority would be nice, but not totally necessary. I've just macro'd my Sacred Shield to spam whisper me when there's 10 seconds remaining... keeps it from every going down. Of course, when things get dicey and I spam the button to make sure it's up, then I get a ridiculous amount of whispers. Other than that, I've been very impressed with the mod so far.
I'm just now getting around to downloading this add-on and so I'm reading through the thread and the macro mentioned here intrigues me. How do you code a macro to tell you when your SS has been up for 50 seconds and then to whisper you? I'm a wow-Luddite so I don't really want to download that mod that tells you when your buff is expiring or whatever that is mentioned in the ret thread and the whispering idea seems great. It's difficult paying attention to action bars, stuff going on around the boss' feet (hello mimiron p4), and then having to peek up at my SS buff and gauge the timer + my potential upcoming free gcd. Thanks for your help !

Offline
Reply With Quote
Old 05/15/09, 10:10 AM   #91
gmedina
Banned
 
Dwarf Paladin
 
Gnomeregan
I like the idea of range check just to let me know when the next ability is out of range. With a lot of bosses requiring constant movment sometimes it is easy to be just out of melee range and not realize it. I would love something that doesn't affect the rotation selection but that maybe highlights the box with red to let you know that you are out of range.

Offline
Reply With Quote
Old 05/15/09, 12:02 PM   #92
Heck
Von Kaiser
 
Heck's Avatar
 
Blood Elf Paladin
 
Velen
Originally Posted by beatific View Post
I'm just now getting around to downloading this add-on and so I'm reading through the thread and the macro mentioned here intrigues me. How do you code a macro to tell you when your SS has been up for 50 seconds and then to whisper you? I'm a wow-Luddite so I don't really want to download that mod that tells you when your buff is expiring or whatever that is mentioned in the ret thread and the whispering idea seems great. It's difficult paying attention to action bars, stuff going on around the boss' feet (hello mimiron p4), and then having to peek up at my SS buff and gauge the timer + my potential upcoming free gcd. Thanks for your help !
I originally used the whisper method I described, but it just ended up being one more thing I had to "look" for. I ended up changing it to an audio cue instead. Much easier to keep track of. I picked sounds your not likely to hear. Macro is as follows:

#showtooltip
/cast Sacred Shield
/script PlaySoundFile("Sound\\Creature\\Herod\\HerodEnrage01.wav")
/in 50 /script PlaySoundFile("Sound\\Creature\\Illidan\\BLACK_Illidan_04.wav")

Also, the mod Power Auras works well also. I just started using that recently. I might not use it for sacred shield, though, because it will be something that's ALWAYS up on my screen and just add to screen clutter... but the mod is great for setting up notices of buffs from encounter mechanics or trinkets or even debuffs (stuff your standing in and need to gtfo of.)

Offline
Reply With Quote
Old 05/15/09, 1:51 PM   #93
Thaeryn
Piston Honda
 
Thaeryn's Avatar
 
Tauren Druid
 
Dethecus
One nice thing I've seen done with Power Auras is to move the aura texture away from the middle of the screen. For example, for SS, you could put a "glow" texture around your character portrait when SS is below 15 seconds. It's still a small amount of screen clutter, but I like having only critical things in the center of my screen.

Also, with regards to clash resolution, the addon actively monitors your cooldown timers and compares them to your current GCD. If two items are expected to come off of cooldown during the same GCD, it will recommend the higher priority. Basically, if you follow the addon exactly, you will maintain the highest dps clash resolutions. It's a fairly simple system, so no need to complicate it.

@beatific - The addon will also recommend SS at the priority you specify if it is below a certain time (user customizable in the options). I understand that on some fights, you might want to change the priority dynamically, which you will want the macro and/or some kind of on-screen notification when below a certain time remaining. Just be aware that the basic functionality is built into the addon already.

I'm keeping tabs on the possible changes to the FCFS rotation that Ghostcrawler has talked about over the last month or so, and if possible, will be updating the addon to work with any changes as soon as they are released. There IS a possibility that the changes could render a large portion of this addon's functionality useless, but if that is the case, I will modify it however necessary to keep it useful.

A bit of a self-plug to leave with. I wrote this addon initially to help our ret paladin test rotations on a target dummy during the 3.1 beta. She liked it so much that I put the time into cleaning it up and releasing it. I now can take a small amount of credit for her dps WMO Paladin DPS Rankings - America (Ceralune - Irregulars)
/bragging

Cheers all.

Offline
Reply With Quote
Old 05/15/09, 1:56 PM   #94
Thorin
Von Kaiser
 
Thorin's Avatar
 
Human Paladin
 
Medivh
Originally Posted by gmedina View Post
I like the idea of range check just to let me know when the next ability is out of range. With a lot of bosses requiring constant movment sometimes it is easy to be just out of melee range and not realize it. I would love something that doesn't affect the rotation selection but that maybe highlights the box with red to let you know that you are out of range.
I love this mod, however it tends to confuse my brain when it shows CS in colour, all this time I think I am in melee range, then I turn to look at my action bar (and CS in red) and I've actually been out of melee range for the last 6 seconds, plumetting my DPS. I know this is part of my responsibility, but you can see where I'm coming from, with the conflicting icons.
Happens often in Kologarn when turning tot he left arm, where melee can be a bit skewy.

Yeah I would love it if the buttons would get coloured red when the ability is out of range, I'm mostly interested in CS.

Great job nonetheless

Offline
Reply With Quote
Old 05/15/09, 3:17 PM   #95
 frmorrison
Protector
 
frmorrison's Avatar
 
Ashstorm
Human Paladin
 
No WoW Account
Originally Posted by Thorin View Post
Happens often in Kologarn when turning tot he left arm, where melee can be a bit skewy.
This doesn't solve on problem other times, but on Kologarn if you just face the chest you can attack either arm (what you see isn't where the targets really are).

Adding a range display is out of the scope of the addon, but changing JoW to JoL would be nice.

Millions of words are written annually purporting to tell how to beat the races, whereas the best possible advice on the subject is found in the three monosyllables: 'Do not try.'

United States Offline
Reply With Quote
Old 05/15/09, 6:08 PM   #96
Thaeryn
Piston Honda
 
Thaeryn's Avatar
 
Tauren Druid
 
Dethecus
Just updated to a new version. Fixed the Spell Priority sub-option menu and added a range check to both the current and next priorities. It was pretty simple to add, and I also added an option to disable it in the options menu. Divine Storm and Consecrate will always show in range. I considered adding an 8yrd range check to those as well, but none of the Paladin abilities are 8 yards, and the closest I can get is a range check on trade distance (yes, the check still works on hostiles). The distance to trade is 11.1 yards though... not close enough to use unfortunately.

You may notice that the range check on the priority icons doesn't update immediately when testing solo or on a target dummy with no one else around. The updates are preformed either on a combat log entry or a target change, so if nothing is happening in the combat log and you move in/out of range, it might appear to be a bit slow. Hardly noticeable for most people when solo, and it won't be an issue at all when grouped. Just an fyi if you see the behavior.

Also, frmorrison, is there any particular reason that you want JoW changed to JoL? I COULD put an option to switch between the two that is selectable in the options menu, but it seems a bit silly for something purely aesthetic. I picked JoW because it contrasted the most with the other abilities, making it less likely to be confused for a CS or DS at a glance. (I still get CS and DS mixed up if I'm not paying attention >.<)

Offline
Reply With Quote
Old 05/15/09, 6:21 PM   #97
 frmorrison
Protector
 
frmorrison's Avatar
 
Ashstorm
Human Paladin
 
No WoW Account
Some addons are used purely for aesthetics!

I prefer JoL's icon because that was the original Judgement icon and I normally Judge Light.

Millions of words are written annually purporting to tell how to beat the races, whereas the best possible advice on the subject is found in the three monosyllables: 'Do not try.'

United States Offline
Reply With Quote
Old 05/16/09, 3:33 PM   #98
Elrendil
Glass Joe
 
Blood Elf Paladin
 
Stormreaver
Originally Posted by Thaeryn View Post
Just updated to a new version. Fixed the Spell Priority sub-option menu and added a range check to both the current and next priorities. It was pretty simple to add, and I also added an option to disable it in the options menu. Divine Storm and Consecrate will always show in range. I considered adding an 8yrd range check to those as well, but none of the Paladin abilities are 8 yards, and the closest I can get is a range check on trade distance (yes, the check still works on hostiles). The distance to trade is 11.1 yards though... not close enough to use unfortunately.
Thanks for the update!

Melee range is 5 yards, so you could do a range check on Crusader Strike and apply it to DS and consecration. It's not totally accurate since it will show red from yards 5-8 but that might not be a big deal.

Offline
Reply With Quote
Old 05/16/09, 7:26 PM   #99
Heck
Von Kaiser
 
Heck's Avatar
 
Blood Elf Paladin
 
Velen
Originally Posted by Elrendil View Post
Thanks for the update!

Melee range is 5 yards, so you could do a range check on Crusader Strike and apply it to DS and consecration. It's not totally accurate since it will show red from yards 5-8 but that might not be a big deal.
Range checks on AoE abilities like DS/Consecrate seem unnecessary.

Offline
Reply With Quote
Old 05/19/09, 7:25 AM   #100
mek
Don Flamenco
 
mek's Avatar
 
Draenei Shaman
 
Tichondrius
Originally Posted by Thaeryn View Post
Also, frmorrison, is there any particular reason that you want JoW changed to JoL? I COULD put an option to switch between the two that is selectable in the options menu, but it seems a bit silly for something purely aesthetic. I picked JoW because it contrasted the most with the other abilities, making it less likely to be confused for a CS or DS at a glance. (I still get CS and DS mixed up if I'm not paying attention >.<)
I feel that there is too much yellow going on in paladin icons already, and I agree the use of Judgement of Wisdom here is probably for the best.

Offline
Reply With Quote
Reply

Go Back   Elitist Jerks » Paladins

Thread Tools

Similar Threads
Thread Thread Starter Forum Replies Last Post
Cat DPS Rotation Kazanir Druids 1356 12/07/09 1:07 PM
Ret FCFS Rotation Helper Thaeryn User Interface and AddOns 0 04/10/09 1:18 AM
FCFS (Retribution) modeling script Left Paladins 25 03/21/09 11:06 AM
Designing a hunter shot-rotation helper addon Zeza Public Discussion 40 11/27/06 8:52 PM