[RET] PVP + Final Verdict
fistenyolo opened this issue ยท 24 comments
Hello,
First and foremost I would like to thank the devs that run this add on it's wonderful!!
I'd like to point out that Final Verdict is not working and only shows POOL. It needs to be a top priority when 3 holy power or more is achieved.
I've attached a Screenshot of Final Verdict not popping up at all once the Legendary is Equipt, I can confirm that Templars Verdict does pop up, but again not made a priority when you have 3 or more Holy Power.
Also, is there a way to remove Divine Storm in PVP, maybe in the Hero Rotation interface in-game checkbox under Retribution?
DS being a priority with 3 or more mobs(PVE) is correct, however in PVP when trying to nuke one guy, you want either Templar's Verdict or Final Verdict to pop up.
Please PM me for a once complete.
Thank you,
Fisten
Does your UI of choice allow you to see spell IDs? Unfortunately, I don't have a level 60 paladin to test, so I can't be sure that I have the correct spell ID for Final Verdict.
337247 is the spell ID of the legendary, which may or may not be the same as the resulting ability on the bar. There's also 336872 (which appears to be the HoW proc associated with FV) and 337228. To confirm, with the legendary equipped, can you please type the following in-game: HeroLib.Spell(337247):IsAvailable()
and press enter (capitalization does matter). If this returns true, then we can confirm that it is indeed the spell ID being used for the on-bar ability. If you could do the same for all three spell IDs, that would be very helpful. Thank you!
One other thing we can try is open your spell book, type /dump "
, then shift-click Final Verdict which should create a link for the spell in chat, add another quotation mark at the end, and press enter. This will return a string similar to this (which is from Enh Shaman's Stormstrike):
|Cff71d5ff|Hspell:17364:0|h[Stormstrike]|h|r
where 17364 is the spell ID.
Perhaps that will work for us? Thank you for being willing to do this testing for us!
I try with 337247 336872 337228 with HeroLib.Spell(337247):IsAvailable() and nothing happens
And thx to you for taking care of this you're doing an amazing work with this add on
I must have not been fully awake when I typed that up. The command would be /dump HeroLib.Spell(337247):IsAvailable()
. Sorry about that.
Hmm, so somehow it's none of them? Just to verify, this was done with the Final Verdict legendary equipped? Also, can you double check 336872? According to the spell data from wowhead, it looks like that's the ID that's supposed to replace TV on the bar. I might have to finish leveling my paladin to get this sorted out.
There's always at least one oddity when it comes to Blizzard programming...
Yes i checked i had the legendary equip and the spell final verdict in my bar and i double check everything
It's working ! Gl man you're good
Last problem is :
Empyrean power talent row 2 is selected addon never see the proc and never display divine storm
Add on doesn't show Divine toll kiryan covenant even if i put main icon/suggested/suggested right
Try the latest files again. This should be fixed in commit 81639aa
For Divine Toll, it should be suggested if either Execution Sentence or Final Reckoning are active on the target while you have 2 or fewer holy power or if...
- You have 0 or 1 holy power AND
- The target does not have the Judgment debuff AND
- Seraphim is active on the target (or that talent is not selected) AND
- Final Reckoning talent is not selected AND
- Execution Sentence talent is not selected or the fight is about to end or there are 5+ targets AND
- Avenging Wrath or Crusade are on cooldown
The APL is very specific about the use condition for Divine Toll.
The profile has been re-written for 9.2 launch. There was previously an issue with FV/TV being skipped if cooldowns were not used as suggested, as spells like Crusade/Execution Sentence/etc being on cooldown were pre-requisites of use. That issue may or may not be resolved with the re-write. If it's not, please let us know.
As far as removing Divine Storm, you can disable AoE suggestions, which should mean it no longer suggests Divine Storm or any other AoE abilities.
Hello,
Problem not solved final verdict not showing at all in rotation instead writing pool or consecration even with all cd up
Add on even select crusader strike when we have 5 holy power
It looks like Simulationcraft replaces Templar's Verdict with Final Verdict behind the scenes, rather than in the APL. I hadn't taken that into account. I've added the substitution in HeroRotation now. Please let me know if it is working after commit 4579cdc
There was a typo in the file. This has been fixed as of commit 6e78ee6
Please copy the latest Retribution.lua and try it again.
I copy the 2 new files and replace the old ones manually cause it seems curse software have not put automatic update yet
When loging paladin it said no rotation supported for this class spe add on disabled
same no rotation find for this spe
maybe i do it incorectly but i reinstalled all package to have a clean set up then i opened in blockpad retribution.lua and clik copy raw content here then save it and close
I pushed version 9.2.0.03. It should show up on Curse soon and will include the updates.
Just Final verdict not showing, i tested without legendary and templar verdict is showing perfectly