So bottom line the behaviour we want to achieve is spam-cast gather 2-3 times before doing the "ascend to drop combat" behaviour?
Yeah - I'm pretty sure that's what it was before (hopefully someone can back this up?). It went in to pick a herb, which aggro'd the mob and interrupted the 1st cast, then it attempted a few more (I think 5 in total(?) and on the 5th interrupt/attempt it resorted to killing the mob or flying away). I cannot remember which, but in hindsight it may have aggro'd 3 or 4 mobs by then so flying away is, in my opinion, the better option than ever having to fight anything. Seeing as there's both the Forager's Gloves and Tauren Druid cast times both equalling 1.5 seconds to herb - that should be enough to have acquired the herb by the 3rd or 4th attempt. Flying away is much better than fighting (due to unsure number of hostile mobs by that point). Atleast that's my opinion anyway.
I think you're right, i've moved some value changes around to hopefully delay triggering the ascend behaviour. Let me know how it runs edit: Has it at any point decided to fight the enemy yet?
Been running it for about 10 minutes - seems to running brilliantly. I witnessed it take a herb in combat, attempt 4 times, grab the herb and fly off successfully. Will keep my eye on it! Hopefully some more people will test this and report back. Once again thanks for the time - I'll report back here if I notice anything. (If I can be that guy - this typo is driving me crazy! [Harvest Helper] - there is herb withing 10 yards range)
Sounds promising Have you seen the aforementioned log message about removing entries from the blacklist yet? Hehe, I'll fix the typo...i remember it annoying the crap out of me as well back when i still had the balls to use GB on my main Thanks for testing and posting results, hope it serves you well
When looking through your last log i noticed the plugin was spamming a lot of exceptions. I tried fixing this as well. I also added a timed blacklist, if the bot fails to pick the herb and decides to abandon it the herb should be blacklisted for 2 minutes. Also...typo fixed!
Will check this new version out! I've had no mentions regarding the "blacklisting" so far. Will keep an eye out for it in this latest ver.
The exception being thrown on pulse kinda indicates that the gathered herbs aren't being added to the safelist, this means they won't be checked against the blacklist and removed from there if they've been mistakenly added to it by GB2 while the plugin "hijacked" control of the character (And since nothing is removed, the aforementioned debug line won't appear). Hopefully the latest fix takes care of this, a good indication is if you see the text "Added currentHerb to latestHarvests" in the debug file for each time a herb has been gathered.
Is this working? I can't seem to see it in my plugins list. I even made a folder for it inside the plugins folder