What's new
By:
  • Visit Rebornbuddy
  • Visit Panda Profiles
  • Visit LLamamMagic
  • Visit Resources
  • Visit Downloads
  • Visit Portal
RebornBuddy Forums

Register a free account today to become a member! Once signed in, you'll be able to participate on this site by adding your own topics and posts, as well as connect with other members through your own private inbox!

[Perdition's Reprieve] - Arms

There appears to be a slight flaw in the CS uptime ticker, but can't work the syntax out.
Line 71 appears to keep casting colossus smash, even if the debuff is up ~1/2 through debuff

if (await S.Cast(SB.spellColossusSmash, !U.colossusSmash)) return true;
 
gaggme said:
There appears to be a slight flaw in the CS uptime ticker, but can't work the syntax out.
Line 71 appears to keep casting colossus smash, even if the debuff is up ~1/2 through debuff

if (await S.Cast(SB.spellColossusSmash, !U.colossusSmash)) return true;
Click to expand...

you can change that line to:

Code: 
if (await S.Cast(SB.spellColossusSmash, !currentTarget.auraExists(SB.spellColossusSmash))) return true;

until he decides to implement a fix
 
gaggme said:
There appears to be a slight flaw in the CS uptime ticker, but can't work the syntax out.
Line 71 appears to keep casting colossus smash, even if the debuff is up ~1/2 through debuff

if (await S.Cast(SB.spellColossusSmash, !U.colossusSmash)) return true;
Click to expand...
I'll need to take a look and add in some logging to verify casting. Personally I don't see this overlap happening but, I have yet to test with Enyo either.
 
greenbrad said:
you can change that line to:

Code: 
if (await S.Cast(SB.spellColossusSmash, !currentTarget.auraExists(SB.spellColossusSmash))) return true;

until he decides to implement a fix
Click to expand...
The call !U.colossusSmash equals the same thing as your post. Are you also seeing this overlap? Could be possible that the cache isn't updating correctly.
 
I can verify it is throwing CS about 1/2 through CS or WB. Will try some more testing testing/tweaking.
 
Found it. Tactics.cs Line 192 for the single target rotation

if (await S.Cast(SB.spellColossusSmash, !U.shatteredDefenses)) return true;

This was causing CS to cast on CD if Shattered Defenses was not up try to force a proc. Changed to this and it was casting at the end of debuff.

if (await S.Cast(SB.spellColossusSmash, !U.colossusSmash)) return true;

Going to try and wrap some more logic around it to cast when the buff is about to fall off, so it hits for added damage & applying same logic to warbringer artifact talent cast. That is unless someone can spell out why this is bad?
 
gaggme said:
Found it. Tactics.cs Line 192 for the single target rotation

if (await S.Cast(SB.spellColossusSmash, !U.shatteredDefenses)) return true;

This was causing CS to cast on CD if Shattered Defenses was not up try to force a proc. Changed to this and it was casting at the end of debuff.

if (await S.Cast(SB.spellColossusSmash, !U.colossusSmash)) return true;

Going to try and wrap some more logic around it to cast when the buff is about to fall off, so it hits for added damage & applying same logic to warbringer artifact talent cast. That is unless someone can spell out why this is bad?
Click to expand...
Currently I am following SimC for the rotation but, I do see your point in an overlap to allow the initial damage to make use of the already existing debuff (assuming it can make use of it). Please let me know what you find to be best and I can create variations if needed.
 
Dagradt ,
The rotation is fine. You need to Refresh CS just Before Mortal Strike to benefit from Shattered Defenses. You dont need to let full duration of CS run. This is how simcraft works and even you should Mortal strike on CD even if you have 1 stack or 2 of Focused Rage.
 
With the nerf to tactician today, going to see how the cards fall. With the reduced procs, may need to keep CS up as long as possible.
 
Any news about this routine ? Actually not bad, but can do better i think ^^ Keep the work it's nice job ;)
 
Public Update:

>> Update: 10/08/2016 <<
​

[Perdition's Reprieve] - Arms will be rolled into [Perdition's Reprieve] - Warrior along with full updates to each spec in accordance with the patchwork level simulations given by SimC and user requests. Please be aware that there may be a brief momentary lapse of time in which none of the Warrior specifications may be available during the transition to the new core.​
 
Dagradt said:
Is it truly that simple? I'd assume this is only useful for initial login and not specialization changes while in the game?
Click to expand...

If you havn't already looked at specialization changes while the bot is on, let me know I can supply you with the code. Didn't really want to copy pasta here though.
 
greenbrad said:
If you havn't already looked at specialization changes while the bot is on, let me know I can supply you with the code. Didn't really want to copy pasta here though.
Click to expand...
I'm using a rather simple method currently but, it may be adding extra weight to the tick process. Please PM me.
 
With charge being a big part of generating rage in down time, could it be possible to add a visual or audio queue for it? Since the CR doesn't handle movement and I haven't quite got the timing down for it either. Thanks
 
Last edited:
You must log in or register to reply here.
Back
Top