Elitist Jerks
Register
Blogs
Forums


Go Back   Elitist Jerks » Class Mechanics » Paladins

Reply
 
LinkBack Thread Tools
Old 04/20/09, 12:36 AM   #46
Enlog
Glass Joe
 
Human Paladin
 
Frostmane (EU)
Great addon Thaeryn, could you add in support for protection paladins please?

Offline
Reply With Quote
Old 04/20/09, 6:54 AM   #47
SwordSa1nt
Von Kaiser
 
SwordSa1nt's Avatar
 
Blood Elf Paladin
 
Neptulon (EU)
Helpful tool, but most of the time it dosen't take in account the global CD when suggesting an ability, or it would suggest it when its still not off cd, and another ability is.

Offline
Reply With Quote
Old 04/20/09, 9:07 AM   #48
Omniro
Glass Joe
 
Tauren Hunter
 
Kargath (EU)
It should factor in global cooldowns (according to code). Maybe you could specify in what situations it recommends what ability instead of what other ability?
It could even be that it doen't factor in haste for the spell-gcds (e.g. exo and conc).

Offline
Reply With Quote
Old 04/20/09, 10:59 AM   #49
Thaeryn
Piston Honda
 
Thaeryn's Avatar
 
Tauren Druid
 
Dethecus
@SwordSa1nt - It should take into account your actual global cooldown. I had previously just used a 1.5 second timer and it caused the predictions to jump around really badly as higher priority abilities came off of cooldown. As of right now, it simply pulls the global cooldown off of your Retribution Aura. If your GCD is hasted, that will show up as well.

@Gergon - Make sure you've updated to the newest version. If you're using an older version that didn't take into account the actual GCD, you could definitely be having a few of those problems. Also, abilities shouldn't pop in as soon as they are available again. You should get some warning as they are coming off of cooldown, at least 1.5 seconds I believe, so if it's blank, you have time to use anything else you may need. If a high priority ability comes off of cooldown, it will replace your current icon with that one, which can sometimes throw you off a little bit. If you hit the ability that was previously recommended, then you are doing it right. If you were doing something else (AoW heal, DP, Salv, SS) then you want to use the new ability that's now showing up. Basically, hit whatever button it shows, and if it happens to change, change the button you're hitting. The logic behind it matches what has been proven in the retribution thread already. You want to try to keep that GCD constantly spinning... don't wait for another ability.

Offline
Reply With Quote
Old 04/20/09, 12:20 PM   #50
Heck
Von Kaiser
 
Heck's Avatar
 
Blood Elf Paladin
 
Velen
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.

Offline
Reply With Quote
Old 04/20/09, 9:37 PM   #51
Elrendil
Glass Joe
 
Blood Elf Paladin
 
Stormreaver
The behavior when all the abilities are on cooldown is a bit funky. It currently displays nothing in the active slot and the next slot when all abilities are on cooldown and then an ability will suddenly pop up with maybe .1 or .2 seconds left on its cooldown so it's tought to react.

I think the ideal thing to do here would be to always show the following abilities in both of these slots. If the next active abilities are coming up in say 2 and 2.2 seconds the icons should be shown with the cooldown shadows reflecting these times.

As some others suggested some extra slots for other common paladin abilities would be great too.

Great idea for a mod by the way.

Offline
Reply With Quote
Old 04/21/09, 12:34 AM   #52
Meranoth
Glass Joe
 
Meranoth's Avatar
 
Human Paladin
 
Shadow Council
I'm still very used to manually watching cooldown and handling clash resolution with a milisecond timer helping me out, but I decided to give this addon a tryout. Hillarious acronym, by the way.

To see the full range of the addon, I disabled my traditional bars (bartender) that had my spells on it and played with the addon. There were a few instances when all abilities were on cooldown and there was no "next cast" slot (as mentioned by the above poster). The other thing I played with was with timing priorities when they would refresh. For example, when an ability finished on its cooldown (say Hammer of Wrath at 1 sec until cooldown completion) and everything but Exorcism was on cooldown (2+ seconds), using Exorcism removed Hammer of Wrath from the "next cast" and "cast now" locations until another ability finished its CD.

This is absolutely an amazing addon for people starting off as a retadin or playing with it as their offspec. One of the most interesting features is how it considers GCD clash on cooldowns.

Offline
Reply With Quote
Old 04/21/09, 8:22 PM   #53
Paul
Glass Joe
 
Paul's Avatar
 
Blood Elf Paladin
 
Proudmoore
I know for a fact that if a higher dps ability is coming off cd .1 seconds from now, I'm going to wait for it. Especially if there's a huge DPS difference. The only time I would hit the ability that's already off cd asap would be if the two abilities were extremely close DPS.

FCFS doesn't always apply.

Offline
Reply With Quote
Old 04/22/09, 1:48 AM   #54
RevokeRepent
Glass Joe
 
Blood Elf Paladin
 
Kilrogg
Just wanted to take the time to say thanks, I've been having a little bit of a rough time adopting to my new FCFS rotation, this thing has been amazing for me.

Offline
Reply With Quote
Old 04/22/09, 10:53 AM   #55
rldolph79
Von Kaiser
 
rldolph79's Avatar
 
Human Paladin
 
Alexstrasza
Originally Posted by Paul View Post
I know for a fact that if a higher dps ability is coming off cd .1 seconds from now, I'm going to wait for it. Especially if there's a huge DPS difference. The only time I would hit the ability that's already off cd asap would be if the two abilities were extremely close DPS.

FCFS doesn't always apply.
You probably could have made an argument pre-3.1 about waiting that extra .1s for judgment to come off cd, but ret abilities are all a lot closer in damage than they used to be. Even pre-3.1 the math showed that waiting even .1s would ALWAYS result in a loss of dps. While you can do whatever you want on your character, it's not a good idea to come around and try to claim the currently accepted best method of dpsing is incorrect without any math to back it up.

Hard math>>>>>>>Opinions

FCFS ALWAYS applies. An extensive amount of time was spent by the authors of the various ret spreadsheets to prove that this is the case. Since you haven't shown anything other than your opinion with no math, it's probably safe to say that everyone else is better off sticking with the method that the math previously proved was best. If, however, you have done the math, please post it in the ret thread, because the math wizards over there would love to see it.

Last edited by rldolph79 : 04/22/09 at 11:00 AM.

Offline
Reply With Quote
Old 04/22/09, 3:41 PM   #56
Thaeryn
Piston Honda
 
Thaeryn's Avatar
 
Tauren Druid
 
Dethecus
Updated.

Did some more work on the jumpiness of the suggestions. I think I've ironed it out pretty well now. Also, added a longer term suggestion when everything is on cooldown. If you use your last ability, and everything else is on cooldown, the icon will go blank only during the GCD to indicate that there are no abilities within the next GCD to recommend. It will then show the next ability to come off of cooldown as the suggestion.

I think I'm going to go ahead and add in both SS and DP as options 7 and 8 in the priority for those people that want it there. I haven't had a lot of time to work on it between work and raiding, but hopefully I can have that, plus a couple of other little things I want to do, sometime in the next week.

There still might be some small issues with the "next" priority icon, especially when all abilities are on cooldown. This is just because of the basic way it's coded (the same algorithm as the current priority - 1.5s for the GCD). I don't particularly watch that icon much, so if it does anything too strange, let me know.

Thanks again for the comments and suggestions.

Thanks for the comments and suggestions.

Offline
Reply With Quote
Old 04/23/09, 1:40 AM   #57
Heck
Von Kaiser
 
Heck's Avatar
 
Blood Elf Paladin
 
Velen
Something happened with the latest build...

Using the agreed upon FCFS priority (CS, HoW, Judgment, DS, Consecrate, Exo) I was having Hammer of Wrath pop up in the middle of my rotations no matter what the mob's heath was at. So if during a fight Judgement, DS, Consecrate or Exo were still on CD then it would show to use Hammer of Wrath until Crusader Strike was ready to be used.

It's really hard to explain what was happening. The disappearing of the UI seemed to be happening less than before, but this seemed to be taking it's place.

Offline
Reply With Quote
Old 04/23/09, 2:02 AM   #58
RevokeRepent
Glass Joe
 
Blood Elf Paladin
 
Kilrogg
Originally Posted by Heck View Post
Something happened with the latest build...

Using the agreed upon FCFS priority (CS, HoW, Judgment, DS, Consecrate, Exo) I was having Hammer of Wrath pop up in the middle of my rotations no matter what the mob's heath was at. So if during a fight Judgement, DS, Consecrate or Exo were still on CD then it would show to use Hammer of Wrath until Crusader Strike was ready to be used.

It's really hard to explain what was happening. The disappearing of the UI seemed to be happening less than before, but this seemed to be taking it's place.
Had the same issue.

Offline
Reply With Quote
Old 04/23/09, 10:54 PM   #59
rea123
Glass Joe
 
Human Warrior
 
Azjol-Nerub (EU)
The HoW issue occurs when all else is on cooldown. The first thing the code does is check for any spells off cooldown, or coming off within one GCD. This test ignores HoW if the spell is unavailable. The next part of the code then looks for the spell with the shortest cooldown of the ones you have prioritised; it doesn't make use of the HoW availability, but since immediately after casting the previous spell HoW gains a GCD cooldown value, it automagically becomes the "soonest available" spell, at least until its GCD expires, or another spell pops into GCD range.

I corrected this by putting the final "HoW soonest" code block in another "howavail == 1" conditional, and that seemed to help; it was then displaying a blank when everything was cooling down, which is better, but not quite right. In addition to this, when determining the spell with the shortest cooldown, each instance of "if <spell> < how" should become "if <spell> < how or howavail == 0".

With these changes in place, that is one seriously sexy addon! That training dummy's gonna hurt tomorrow...

EDIT: I cocked up the cooldown fix on the forum - rather than "and howavail == 1", it should read "or howavail == 0". The correct version is what I tested against; the incorrect version is merely what I posted to the world (now fixed).

EDIT the second: Removed suggestion to mail me for updated file - was intended for author, not general use; I have no right to be distributing modified versions of the addon.

Last edited by rea123 : 04/24/09 at 6:58 AM. Reason: Faulty code

Offline
Reply With Quote
Old 04/24/09, 11:46 AM   #60
Thaeryn
Piston Honda
 
Thaeryn's Avatar
 
Tauren Druid
 
Dethecus
Thanks for looking into that rea123. I noticed that error after I had started adding in new priorities for DP and SS. I've also added a couple of other configuration options for showing the mod on viable targets, always while in combat, or only on boss targets.

I've ran into a couple of other bugs so I don't have the new version ready to go just yet, but I'll hopefully be releasing it later today or tomorrow with the HoW fix as well. Sorry about letting that one slip through.

EDIT:

Finished the update. Here are the changes/fixes:

- Added 2 more priority slots and support for Divine Plea and Sacred Shield.
- Added conditional options for mana to use Divine Plea and time left on Sacred Shield before refreshing.
- Removed "blank" spot in the rotation suggestion. Suggestions should now be very fluid with almost no jumpiness. The only jumps that I found occured when you were out of GCD (as another, higher priority, ability comes off of cooldown) or when you have been below the mana threshold for Consecrate and you rise above it, making Consecrate instantly available.
- Fixed calculations for Hammer of Wrath. It now does an additional check for target health before considering it for a long term suggestion.
- Added "Show On" option, allowing you to only show the addon on a "Viable Target" (the previous behavior), while "In Combat" (even if not targeting anything), and on "Bosses Only" (any mob with a skull instead of a level).
- Split the configuration screen into a main configuration page and a priority sub-page.
- Fixed usage at lower levels. Once you have Divine Storm, you should be able to use this addon, as long as you don't set a priority for a spell you don't know (Sacred Shield, Divine Plea).

A couple of things I haven't been able to test extensively are the "In Combat" option and using the addon while off-spec'd as Prot or Holy. If you run into any issues, please let me know (preferably via PM if just reporting an error message).

Thanks and enjoy.

Last edited by Thaeryn : 04/24/09 at 6:12 PM.

Offline
Reply With Quote
Reply

Go Back   Elitist Jerks » Class Mechanics » Paladins

Thread Tools

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