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

[Warlock CC] skiWarlock 5.0 -Ok, probably doesn't work right now-

Status
Not open for further replies.
ski said:
Indeed. Been working on getting my lock up to new raiding gear :P

And botting a rogue. I don't believe the combat issue is something I can fix on my end though.
Click to expand...

Which? The fact that it loops the "rest" stuff when you use Mr. AutoFight, etc? Would it be possible for you to talk with the maker of that add-on as he swears that it isn't anything on his end?

I'm about to try and delve into it and try and rip out the "rest" code from that addon, but I'm sure I'll mess something up. =)

Glad to see that you're having fun with the game, tho.
 
Okay... Playing around with it some more there's something wonky with the pet detection code. I disabled the loop that does the "at rest" section of Mr. AutoFight so that's not triggering at all.

But after many of the fights (especially the last one) the CC is swearing that I need a pet... Even tho she's right there next to me. As soon as I walk away from the training dummy it tells me this:

Code: 
[10:51:02 PM:36] [skiWarlock] Combat: Pet Attack: Training Dummy
[10:51:02 PM:75] [skiWarlock] Combat: Affliction: Lifetap
[10:51:02 PM:83] Spell_C::CastSpell(1454, 0, 0xF13079A800006AD9, 0)
[10:51:03 PM:207] Activity: Combat
[10:51:03 PM:232] [skiWarlock] Combat: Pet Attack: Training Dummy
[10:51:03 PM:293] [skiWarlock] Combat: Affliction: Shadow Bolt
[10:51:03 PM:303] Spell_C::CastSpell(686, 0, 0xF13079A800006AD9, 0)
[10:51:04 PM:412] Activity: Combat
[10:51:04 PM:429] [skiWarlock] Combat: Pet Attack: Training Dummy
[10:51:04 PM:480] [skiWarlock] Combat: Affliction: Haunt
[10:51:04 PM:491] Spell_C::CastSpell(48181, 0, 0xF13079A800006AD9, 0)
[10:51:05 PM:799] Activity: Combat
[10:51:05 PM:814] [skiWarlock] Combat: Pet Attack: Training Dummy
[10:51:05 PM:874] [skiWarlock] Combat: Affliction: Haunt
[10:51:05 PM:886] Spell_C::CastSpell(48181, 0, 0xF13079A800006AD9, 0)
[10:51:07 PM:319] Activity: Combat
[10:51:07 PM:340] [skiWarlock] Combat: Pet Attack: Training Dummy
[10:51:07 PM:398] [skiWarlock] Combat: Affliction: Unstable Affliction
[10:51:07 PM:406] Spell_C::CastSpell(30108, 0, 0xF13079A800006AD9, 0)
[10:51:08 PM:420] Activity: Combat
[10:51:08 PM:449] [skiWarlock] Combat: Pet Attack: Training Dummy
[10:51:08 PM:555] [skiWarlock] Combat: Affliction: Bane of Agony
[10:51:13 PM:463] Spell_C::CastSpell(980, 0, 0x0, 0)
[10:51:24 PM:987] [skiWarlock] NeedRest: Need Pet

Thoughts?
 
Can you tell me how to change pet from sucubuss to void for affliction :)?
 
I'v been running this CC from 1-30+ and this one works just FLAWLESS, the only bug that i'v got is targetting issues when in an instance and sometimes it doesnt detect adds that accidently aggrod my Pet :P Other then that, AWSOME work <3
 
I just noticed that it doesn't seem that any of the >80 spells are coded in yet. Any way to get even just FelFlame entered in (since it is useful to at least 2 specs) to the rotation?
 
- Your Spec demo
- Your Level 46
- Your Log
- What happened not sure keep spiting this in the debug
[8:59:11 AM:546] [skiWarlock] Combat: Pet Attack: Fire Roc
[8:59:11 AM:718] System.NullReferenceException: Object reference not set to an instance of an object.
at Styx.Bot.CustomClasses.skiWarlock.demonologyDecideSpell(String logPrefix) in c:\hb\CustomClasses\skiWarlock\skiWarlock.cs:line 1782
at Styx.Bot.CustomClasses.skiWarlock.SingleRotation(String logPrefix, WoWUnit Mob) in c:\hb\CustomClasses\skiWarlock\skiWarlock.cs:line 2994
at Styx.Bot.CustomClasses.skiWarlock.Combat() in c:\hb\CustomClasses\skiWarlock\skiWarlock.cs:line 1189
at Bots.Grind.LevelBot.a7(Object A_0)
at TreeSharp.Action.RunAction(Object context)
at TreeSharp.Action.a.a()
at TreeSharp.Composite.Tick(Object context)
at TreeSharp.Sequence.a.b()
at TreeSharp.Composite.Tick(Object context)
at TreeSharp.PrioritySelector.a.b()
at TreeSharp.Composite.Tick(Object context)
at TreeSharp.PrioritySelector.a.b()
at TreeSharp.Composite.Tick(Object context)
at TreeSharp.Decorator.a.a()
at TreeSharp.Composite.Tick(Object context)
at TreeSharp.PrioritySelector.a.b()
at TreeSharp.Composite.Tick(Object context)
at TreeSharp.PrioritySelector.a.b()
at TreeSharp.Composite.Tick(Object context)
at TreeSharp.PrioritySelector.a.b()
at TreeSharp.Composite.Tick(Object context)
at Styx.Logic.BehaviorTree.TreeRoot.b()
- The situation in which it happened (so I can recreate it) also not sure I am running the bot in tanaris killing fire roc Glasshide Basilisk Duneclaw Burrower and Basking Cobra
- Any other details that you think may help me fix it
 

Attachments

I changed the code that deals with CoE to make sure it doesn't cast CoE on a target that already has the Jinx talent effect (makes CoE an AE effect). That way it saves some mana and doesn't waste a GCD.

Changed:
Code: 
if (Me.CurrentTarget.HealthPercent > Settings.DotMinTargetHealth  && !Me.CurrentTarget.HasAura("Curse of the Elements") &&  Settings.UseCoE && CanCastOn("Curse of the Elements",  Me.CurrentTarget) && LastSpell.Key != "Curse Of the Elements")

To:
Code: 
 if (Me.CurrentTarget.HealthPercent > Settings.DotMinTargetHealth  && !Me.CurrentTarget.HasAura("Curse of the Elements") &&  !Me.CurrentTarget.HasAura("Jinx: Curse of the Elements") &&  Settings.UseCoE && CanCastOn("Curse of the Elements",  Me.CurrentTarget) && LastSpell.Key != "Curse Of the Elements")

I'm using it right now and it works great.
 
I'll be going through the CC and making updates/fixes/new spells in the next week or so. It is the holidays and I'm moving into the new house so I'm pretty busy, but I'll try and get it out by next week.
 
ski said:
I'll be going through the CC and making updates/fixes/new spells in the next week or so. It is the holidays and I'm moving into the new house so I'm pretty busy, but I'll try and get it out by next week.
Click to expand...

<3 So many times I opened up your source and began tweaking ... but it just hurts my head lol :-P.
 
Thanks ski for the work. I hope you can get this to work with mr auto fight aswell.
 
Densdale said:
Thanks ski for the work. I hope you can get this to work with mr auto fight aswell.
Click to expand...

Er... It does work with Mr Auto Fight. I was using it earlier for questing in Uldum.

Ski Warlock v5.26 (Last Updated: 2:56 PM 10/08/2010)
 
awesome cc, getting about about 20% more exp (lvl 43 now), then with the default class
 
Scattered said:
Er... It does work with Mr Auto Fight. I was using it earlier for questing in Uldum.

Ski Warlock v5.26 (Last Updated: 2:56 PM 10/08/2010)
Click to expand...

It does, but there are weird issues with Mr. Auto Fight (and the Combat/Healing bot) when it will loop the rest state and try to re-summon the Demon infinitely as well as get stuck on summoning the Healthstone.

It doesn't happen all the time, but it does happen.

The one nagging bug that REALLY screws things up for me is that if I'm attacking a single critter, the CC will sometimes target and send the pet off to attack another target that's in my character's direct line of sight after it kills the *solo* critter. So if I don't watch it in a dungeon the pet will go and aggro another group if there's other NPC's close to where we were just fighting (but are not agroing).
 
double casting UA

hi guys,
anyone else getting an occassional double casting of unstable affliction? it happens 25%-33% at a time. If there is a fix for this pls link or let me know ^_^ THanks!
 
It doesn't look like the new Warlock ability "Soul Swap" is supported in here: I was trying to implement it but I see in the code it's using the object SpellManager - is this where the addition of a new spell would have to happen in order to trigger it from the CC?

Thanks!
 
Status
Not open for further replies.
Back
Top