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

[BUG] 1.7.1.5 Monk No Longer Using Blinding Flash

W

whitefang

New Member
Joined
Nov 27, 2012
Messages
23
Reaction score
0
Monk is using blinding flash only for sweeping winds now. With 4 set Inna bonus sweeping is rarely needing to be recast. I've watched my monk kill 10+ elite packs now without using blinding flash on them once. This is a massive DPS loss.
 
whitefang said:
Monk is using blinding flash only for sweeping winds now. With 4 set Inna bonus sweeping is rarely needing to be recast. I've watched my monk kill 10+ elite packs now without using blinding flash on them once. This is a massive DPS loss.
Click to expand...

I have try with Inna set, Bf is used for SW and while combat with Elite.
 
hti said:
I have try with Inna set, Bf is used for SW and while combat with Elite.
Click to expand...

Only if the first thing you fight is an elite pack. Once SW is up it never casts BF again unless SW falls off.
 
whitefang said:
Only if the first thing you fight is an elite pack. Once SW is up it never casts BF again unless SW falls off.
Click to expand...

No, I have try it just before reply, my monk cast it on white mob before enabling SW (BH is also cast now : 15% DPS more in SW) and cast on elite normaly when he fight.
I can make more test tomorow, now I go sleep a little :D
 
I can confirm BF does not fire as it should after doing the swap. I'll look at the code to see why, but definitely does not use it properly with 1.7.1.6.

EDIT - for folks not doing the weapon swap it probably works fine. For those of us who do have it, it will only fire upon the swap OR if we have to use it defensively. Needs to be addressed.
 
Last edited:
This should fix it...working for me:

Change this:
Code: 
if (!Settings.Combat.Monk.SweepingWindWeaponSwap || !weaponSwap.CanSwap())
                    return new GilesPower(SNOPower.Monk_BlindingFlash, 0f, vNullLocation, iCurrentWorldID, -1, 0, 1, USE_SLOWLY); //intell -- 11f -- 1, 2

to this:
Code: 
if (!Settings.Combat.Monk.SweepingWindWeaponSwap || !weaponSwap.CanSwap() || (weaponSwap.CanSwap() && CheckAbilityAndBuff(SNOPower.Monk_SweepingWind)))
                    return new GilesPower(SNOPower.Monk_BlindingFlash, 0f, vNullLocation, iCurrentWorldID, -1, 0, 1, USE_SLOWLY); //intell -- 11f -- 1, 2


EDIT: I've submitted a merge request via GIT for to Unified Trinity. Has been working well for me.
 
Last edited:
Magi said:
This should fix it...working for me:

Change this:
Code: 
if (!Settings.Combat.Monk.SweepingWindWeaponSwap || !weaponSwap.CanSwap())
                    return new GilesPower(SNOPower.Monk_BlindingFlash, 0f, vNullLocation, iCurrentWorldID, -1, 0, 1, USE_SLOWLY); //intell -- 11f -- 1, 2

to this:
Code: 
if (!Settings.Combat.Monk.SweepingWindWeaponSwap || !weaponSwap.CanSwap() || (weaponSwap.CanSwap() && CheckAbilityAndBuff(SNOPower.Monk_SweepingWind)))
                    return new GilesPower(SNOPower.Monk_BlindingFlash, 0f, vNullLocation, iCurrentWorldID, -1, 0, 1, USE_SLOWLY); //intell -- 11f -- 1, 2


EDIT: I've submitted a merge request via GIT for to Unified Trinity. Has been working well for me.
Click to expand...

great magi, gonna try it right now :)

edit: working fine!!! i have also changed this part:

Code: 
            // Blinding Flash
            if (!bOOCBuff && playerStatus.CurrentEnergy >= 20 && hashPowerHotbarAbilities.Contains(SNOPower.Monk_BlindingFlash) &&
                (
                    iElitesWithinRange[RANGE_15] >= 1 || (iAnythingWithinRange[RANGE_20] >= 5 && iElitesWithinRange[RANGE_50] == 0) ||
                    (CurrentTarget.IsBoss && CurrentTarget.RadiusDistance <= 15f) ||
                    (iAnythingWithinRange[RANGE_15] >= 1 && hashPowerHotbarAbilities.Contains(SNOPower.Monk_SweepingWind) && !GilesHasBuff(SNOPower.Monk_SweepingWind) && Settings.Combat.Monk.HasInnaSet)

with this it only casts on elite or 5+ mobs in near distance and no elite in far distance ;-)
 
Last edited:
Nice! I'll pop that in my code and give it a whirl as well.
 
I set it this way on purpose as the constant dmg from sweeping winds is way more effective than the 3 second buff you will get during combat.
Meaning if you lose sweeping winds and blinding flash is on cool down... You lose a lot of dmg or you wait a long time until cast.
 
tesslerc said:
I set it this way on purpose as the constant dmg from sweeping winds is way more effective than the 3 second buff you will get during combat.
Meaning if you lose sweeping winds and blinding flash is on cool down... You lose a lot of dmg or you wait a long time until cast.
Click to expand...

But SW is basically always up so...more dps with our fix ^^

end in those 3sec I down the elites in A3 MP3-4 ;-)
 
blubby said:
But SW is basically always up so...more dps with our fix ^^

end in those 3sec I down the elites in A3 MP3-4 ;-)
Click to expand...

Ok let me explain this simply:

If swapping is done correctly each cyclone should tick for 30k with crits over 100k - 2 ticks every second.
Meaning AVG of 120k DPS from each cyclone.
Sweeping winds around you does 45% not 20% meaning it will do X2 more than each cyclone meaning OVER 240DPS.
I attack at 2.3attacks per second with 50% crit chance making it create an avg of 1.15*(Chance to pop a cyclone - which is not known but is prolly high)*Number of monsters hit from every punch.
Meaning 1Cyclone + Sweeping winds dmg = 360k DPS (remember if it is 1 cyclone per second and each cyclone lasts 3 seconds then you allways have 3 up) -> 3Cyclones + Sweeping winds = 600K Dps.
Meaning that running with 100K dps on yourself is a waste of time to start with... and that 30% bonus is useless even more - especially if you lose sweeping winds buff AND blind is on cooldown.

But do as you wish, these are just my 2c from trying many different builds with this setup.
 
Since applying this fix I've noticed a decent increase in IPH and XP/hr. The chances if SW falling off when you have 4 piece Inna's and not having BF ready to recast are very low. It would be nice to have the SW routine wait until BF is ready to recast though.
 
blinding flash is still quite beneficial to have. sweeping winds is pretty much constantly up for inna users so it doesn't make sense not to use it when fighting elites (and especially ubers!). If you fight on high MP levels like myself, the crowd control benefit bf gives you is quite helpful (dps doesn't hurt either).

It's also a brief cool down so I don't think the wait poses a problem for non-inna's but I'll let them decide. I definitely saw a negative impact on my monk with it not casting (and I've got Inna and use your swap). I run MP5-6 though so crowd control is important.
 
whitefang said:
Since applying this fix I've noticed a decent increase in IPH and XP/hr. The chances if SW falling off when you have 4 piece Inna's and not having BF ready to recast are very low. It would be nice to have the SW routine wait until BF is ready to recast though.
Click to expand...

It will wait for blinding flash's cool down before attempting swap/SW.
 
That's very nice! I see no reason not to at least use BF on every elite pack.
 
My fix has already been merged into Trinity Developer Branch so it should be released with next Trinity update. I've got to test blubby's code and than I can send that merge request over to rrrix as well.
 
Magi said:
My fix has already been merged into Trinity Developer Branch so it should be released with next Trinity update. I've got to test blubby's code and than I can send that merge request over to rrrix as well.
Click to expand...

Sweet. They should be Ok just to drop straight over 1.7.1.6 though shouldn't they ?

G
 
You must log in or register to reply here.
Back
Top