gust slash problem?

I can't seem to get the bot to use gust slash as a combo with spinning edge with both magitek and kupo.

Magitek code:

Code:

Casting.Cast(r => "Gust Slash", r => Actionmanager.LastSpell.Name == "Spinning Edge", r => Core.Player.CurrentTarget, r => 3, r => false),

Kupo code:

Code:

Spell.Cast("Gust Slash", r => Actionmanager.LastSpell.Name == "Spinning Edge"),

Is my code wrong?

 

If I remember my Magitek formatting correctly it should be:

Code:

return new PrioritySelector(
			Casting.Cast(r => "Gust Slash", r => Actionmanager.LastSpell.Name == "Spinning Edge", r => Core.Player.CurrentTarget, r => 2, r => false),
			Casting.Cast(r => "Spinning Edge", r => true, r => Core.Player.CurrentTarget, r => 2, r => false));

 

Oh snap I just posted this exact same issue here:

https://www.thebuddyforum.com/rebor...rt/184705-rogue-ninja-combat.html#post1696951

Mine has the whole Kupo script and log attached, if that helps. Having the exact same issue as the OP. Glad I'm not crazy. (I'm NOT a programmer, though)

 

I'm editing the allbasicroutine file, where do i put that code Endus? It's strange though, i tried allbasicroutine with gladiator and it does the combo perfectly...

 

Yeah, I'm guessing something is wrong. Like the bot isn't able to detect that the last move you used was Spinning Edge, so that always returns false. I know very little about this, though, just using common sense. I could be entirely wrong. Hopefully a dev of some kind can confirm.

Obviously this is brand new so problems are expected.

 

Yeah, I'm starting to think that way too. I'm no developer, that's for sure, but comparing it to other code, it seems to look spot on, so i guess we will wait for someone more knowledgeable to give us a hand.

 

Ahh I forgot Magitek has a funky setup for Classes under lvl 30 (I'm used to working with completely different .cs files for the Class and the Job). Sorry for confusing you.

Code:

//Level 4
Casting.Cast(r => "Gust Slash", r => Actionmanager.LastSpell.Name == "Spinning Edge", r => Core.Player.CurrentTarget, r => 3, r => false),

//Level 1
Casting.Cast(r => "Spinning Edge", r => true, r => Core.Player.CurrentTarget, r => 3, r => false),

There's lots more code that needs to be changed in Magitek in order for it to recognize the Class/Job skills though. That just tells it how to use it. If I can't get MoreDots working properly tomorrow (and no other CRs pop up before then) I might take a swing at Magitek.

 

The bot doesn't seem to register Actionmanager.LastSpell.Name for the rogue skills currently.

I tried logging the result for LastSpell and it came out "Empty Spell" instead of Spinning Edge, etc.

Other class seems fine.

I did a work around by registering the spells manually by using the DoubleCastDict but the downside is it can't register whether you've missed the opening combo or not. So it will always go Spinning Gust Dancing even if you've missed the Spinning Edge and Gust Slash.

I'm sure mastahg'll fix these soon.

 

Thanks for that neko, it's good to see you back here again. Can't wait until he releases a fix.

 

Can't you use a sequence instead :S

 

...
...because I didn't remember about sequences until you mentioned it?

Well, I was kind of hoping that the DoubleCastDict won't register missed attacks but yeah, should of gone with sequence :S

 

What does sequence code look like?

 

for trees: return new Sequence( //with Decorators, selectors or w/e

for coroutines: await a && await b & await c

 

I'm no coder so most of that looks foreign to me . Any examples of how I could use that in the Magitek routine?

 

I don't know magitek routine and looking at it would just slow down coding of my routine

I finally finished it's core and I'm starting rotation today so hopefully by tonight I'll post it.

Warning though, my account is brand new so I won't be able to test abilities and logic to rotation.

 

change

Code:

                Casting.Cast(r => "Gust Slash", r => Actionmanager.LastSpell.Name == "Spinning Edge", r => Core.Player.CurrentTarget, r => 3, r => false), //L4

to 

                Casting.Cast(r => "Gust Slash", r => Actionmanager.LastSpellId == 192, r => Core.Player.CurrentTarget, r => 3, r => false), //L4

if that doesnt work, use

Code:

                Casting.Cast(r => "Gust Slash", r => Actionmanager.LastSpellId == 2240, r => Core.Player.CurrentTarget, r => 3, r => false), //L4

 

It actually got fixed. LastSpell now recognizes "Spinning Edge"

 

spellid >_>

I'm guessing Actionmanager.DoAction(spellname, target) still breaks on foreign languages?
Or did they add a dictionary into core for non-english clients to use english spellnames? XD

 

As far as I know strings will still break other languages (so if you're interested in multi-language usage you're forced to go the ID route).

 

It looks like the problem is solved now, thank you all for your input and help