Elitist Jerks
Register
Blogs
Forums


Go Back   Elitist Jerks » Class Mechanics » Paladins

Reply
 
LinkBack Thread Tools
Old 10/03/12, 10:09 AM   #886
slipey
Glass Joe
 
Human Paladin
 
Stormreaver
I used the latest alpha in raid last night and everything was working great. It handled T90 talent switches without issue when both LH and ES were included in my prio string. This is just what the doctor ordered! Thanks for the usability update.

Offline
Reply With Quote
Old 10/03/12, 3:35 PM   #887
Kragshot Prime
Glass Joe
 
Kragshot Prime's Avatar
 
Human Paladin
 
Shu'halo
I haven't seen this mentioned, but for me; CLCinfo is not automatically detecting and shifting according to spec change between ret and prot. I installed the latest alpha and it's still not working.

Has that been abandoned with the current builds? If so, then I apologize for missing it; but if it hasn't (and I don't see why it would have been), then I'm putting this out there.

And for the record, I'm no script-jock so, unless there's a simple fix; just let me know when it's been addressed.

Otherwise, still an awesome app.

Offline
Reply With Quote
Old 10/03/12, 7:21 PM   #888
Jourgenson
Glass Joe
 
Jourgenson's Avatar
 
Blood Elf Paladin
 
Draenor
Originally Posted by Kragshot Prime View Post
I haven't seen this mentioned, but for me; CLCinfo is not automatically detecting and shifting according to spec change between ret and prot. I installed the latest alpha and it's still not working.

Has that been abandoned with the current builds? If so, then I apologize for missing it; but if it hasn't (and I don't see why it would have been), then I'm putting this out there.

And for the record, I'm no script-jock so, unless there's a simple fix; just let me know when it's been addressed.

Otherwise, still an awesome app.
You probably have the "Force template regardless of spec" option enabled in the template options. Set this disabled and make sure your Ret and Prot templates have the correct class specialization and distinctive talents set.

“All parts should go together without forcing. You must remember that the parts you are reassembling were disassembled by you. Therefore, if you can’t get them together again, there must be a reason. By all means, do not use a hammer.”
-IBM Manual, 1925

Canada Offline
Reply With Quote
Old 10/07/12, 10:24 AM   #889
climaximus
Glass Joe
 
Blood Elf Paladin
 
Cho'gall
Originally Posted by burghy View Post
I'll look into the talents issue. A way to bypass it for now is to copy your current ret template a bunch of times and make it work for specific talents.

LE:
New clcinfo alpha should be available soon with following changes:
- detects exorcism glyph so mexo not needed
- checks if the talent is available for sacred shield and the lvl 90 talents and shouldn't try to use them if not (the idea would be to list everything and the ones not available to be ignored)
- also added inq version of all lvl 90 talents

Pretty tired atm so let me know if everything works as intended.
Will you be adding these to CLCRET as well?

United States Offline
Reply With Quote
Old 10/07/12, 11:26 AM   #890
burghy
Don Flamenco
 
Human Paladin
 
Ravencrest (EU)
Should be already implemented since r205.

Offline
Reply With Quote
Old 10/10/12, 12:05 PM   #891
climaximus
Glass Joe
 
Blood Elf Paladin
 
Cho'gall
The automatic detection is not working for me when I change my L90 talent to LH. It still shows ES in the priority system when I change my L90 talent to LH. When I have ES selected as my L90 talent, it does not show LH in the priority so that's working properly. Am I doing something wrong? This is the string I'm using:

inq inqdp tv5 inqes lh how exo cs j tv3 ss

Thanks

United States Offline
Reply With Quote
Old 10/12/12, 12:38 AM   #892
Savante
Glass Joe
 
Blood Elf Paladin
 
Dragonmaw
What's the *current* Best string to use? Thanks gang!

Offline
Reply With Quote
Old 10/12/12, 2:00 AM   #893
burghy
Don Flamenco
 
Human Paladin
 
Ravencrest (EU)
@climaximus can't reproduce it with that string, even with es as first in priority it still doesn't show at all.

Offline
Reply With Quote
Old 10/12/12, 3:16 AM   #894
aylen86
Piston Honda
 
aylen86's Avatar
 
Human Paladin
 
Malygos (EU)
Originally Posted by climaximus View Post
The automatic detection is not working for me when I change my L90 talent to LH. It still shows ES in the priority system when I change my L90 talent to LH. When I have ES selected as my L90 talent, it does not show LH in the priority so that's working properly.
It works fine for me. Which version of clcret/clcinfo do you use?

Offline
Reply With Quote
Old 10/15/12, 2:02 PM   #895
climaximus
Glass Joe
 
Blood Elf Paladin
 
Cho'gall
Originally Posted by aylen86 View Post
It works fine for me. Which version of clcret/clcinfo do you use?
Curse says I'm up to date. How do I check in game what version it's running?

United States Offline
Reply With Quote
Old 10/16/12, 11:21 AM   #896
Ronark
Don Flamenco
 
Ronark's Avatar
 
Blood Elf Paladin
 
Hyjal
Originally Posted by climaximus View Post
Curse says I'm up to date. How do I check in game what version it's running?
You should beable to check in the Addon loader in the character select screen, or look in the Lua file itself, there is usually a version declaration there.

Dice you have the curse client, you can also look in the Description box or the changelog which should list each version number.

United States Offline
Reply With Quote
Old 11/06/12, 8:23 AM   #897
MrPaladinGuy
Glass Joe
 
MrPaladinGuy's Avatar
 
Human Paladin
 
Stormrage
Originally Posted by climaximus View Post
The automatic detection is not working for me when I change my L90 talent to LH. It still shows ES in the priority system when I change my L90 talent to LH. When I have ES selected as my L90 talent, it does not show LH in the priority so that's working properly. Am I doing something wrong? This is the string I'm using:

inq inqdp tv5 inqes lh how exo cs j tv3 ss

Thanks
Originally Posted by aylen86 View Post
It works fine for me. Which version of clcret/clcinfo do you use?
I've had this same issue for a while now using CLCRet 1.5.04.003

Inputting LH for any reason causes it to remain in the rotation regardless of which rotation I'm using even if LH is no longer in the rotation.

If I chose another T6 talent CLCRet proceeds to recommend whichever Tier 6 talent I'm using in it's place regardless of the rotation I input.

Example - I can input just CS and J which CLCRet accepts, but it will continue to recommend whichever T6 talent I have as if it was still running off the initial rotation that was added that included LH.

It also might be worth noting that whichever T6 talent I chose CLCRet will show that as the current ability if It's not on cooldown.

I've tried the basic fixes including the removal of the mod and all related files and trying it again.

I will provide a video shortly after servers are up.

Last edited by MrPaladinGuy : 11/06/12 at 1:34 PM. Reason: To be more specific

United States Offline
Reply With Quote
Old 12/06/12, 3:51 PM   #898
slipey
Glass Joe
 
Human Paladin
 
Stormreaver
I was hoping you could make a few enhancements. Currently the Simulationcraft default actionlists use a few actions that aren't possible with the current CLCInfo.

1. First is a HoW clash detection. Similar to the old Judgment clash option, I would like to be able to delay my next ability by 0.2-.04 if it would allow HoW to be next. The Simulationcraft action is:

actions+=/hammer_of_wrath
actions+=/wait,sec=cooldown.hammer_of_wrath.remains,if=cooldown.hammer_of_wrath.remains>0&coold own.hammer_of_wrath.remains<=0.2

I hacked this into the HoW action in my local copy as seen below:

how = {
		id = howId,
		GetCD = function()
			if IsUsableSpell(howId) and s1 ~= howId then
				return max(0, GetCooldown(howId) - 0.2)
			end
			return 100 -- lazy stuff
		end,
2. I'd also like a new action for Judgment which checks for HoW being available. The Simulationcraft action is:

actions+=/judgment,if=target.health.pct<=20|buff.avenging_wrath.up

Something like:

jhow = {
		id = jId,
		GetCD = function()
			if IsUsableSpell(howId) and (s1 ~= jId) then
				return GetCooldown(jId)
			end
			return 100 -- lazy stuff
		end,
		UpdateStatus = function()
			s_ctime = s_ctime + s_gcd + 1.5
			if s_ha > 0 then
				s_hp = min(5, s_hp + 3)
			else
				s_hp = min(5, s_hp + 1)
			end
		end,
		info = "Judgement while HoW is available",
	},
Let me know what you think. Thanks again for all the hard work.

Offline
Reply With Quote
Old 12/07/12, 4:26 AM   #899
aylen86
Piston Honda
 
aylen86's Avatar
 
Human Paladin
 
Malygos (EU)
I would greatly appreciate that. In fact Sanctified Wrath forces us to use an optimal priority sequence which is not implemented in clcinfo yet.

Offline
Reply With Quote
Old 12/07/12, 6:43 AM   #900
burghy
Don Flamenco
 
Human Paladin
 
Ravencrest (EU)
New alpha should be up soon, let me know if the requested stuff works as intended.

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