kolton / d2bot-with-kolbot

d2bot game manager by D3STROY3R with kolbot libs by kolton for d2bs
345 stars 332 forks source link

Barb dont switch weapons while whirlwind #266

Open yharahuts opened 7 years ago

yharahuts commented 7 years ago

So I have barb with

Config.TeleSwitch = true;
Config.BOSwitch = 0;
Config.FindItem = true; 
Config.FindItemSwitch = 1;

MF switch is off:

Config.MFSwitchPercent = 0; // Boss life % to switch weapons at. Set to 0 to disable.
Config.MFSwitch = 0; // MF weapon slot: 0 = slot I, 1 = slot II

For some reason it doesnt switch to first slot before attacking, and whirlwinding thru the enemies with precast slots. Any ideas why?

As a temp fix I added this to ClassAttack.whirlwind:

Precast.weaponSwitch(0);
poirierdev commented 7 years ago

Does the inverse work? Maybe the slot was hard-coded in the underlying logic.

On Thu, Aug 31, 2017 at 7:00 AM, Legless notifications@github.com wrote:

So I have barb with

Config.TeleSwitch = true; Config.BOSwitch = 0; Config.FindItem = true; Config.FindItemSwitch = 1;

For some reason it doesnt switch to I slot before attacking, and whirlwinding thru the enemies with precast slots. Any ideas why?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/kolton/d2bot-with-kolbot/issues/266, or mute the thread https://github.com/notifications/unsubscribe-auth/ALT8rHLWDomW86v3_0GCHV8Qw-9ZrLKKks5sdoQ1gaJpZM4PImY6 .

StoneOfJordan commented 7 years ago

Is it possible that you joined the game with slot 1 active? (0 being your attack slot)

yharahuts commented 7 years ago

@StoneOfJordan I'm not sure. It was happened 2 times in diffirent games. After merc killed all the pack (coz barb have like 200 dmg), barb looted it, switched to slot I and everything were ok after.

I will turn off my fix and look into it.

yharahuts commented 7 years ago

Just saw it again @ pit.

Barb killed a monsters pack, looted switched to II slot, looted them, then tele'd to another pack and killed it without switching to slot I. Still not sure why it did so.

c7hrozq

StoneOfJordan commented 7 years ago

I see, the cause is the way it's implemented. You'll have to add some additional checks to the Pather.js for it to work properly for area clearing.

mcvk56 commented 5 years ago

any concrete solution to this problem?