• Visit Rebornbuddy
  • gust slash problem?

    Discussion in 'Rebornbuddy Forum' started by exaccuss, Oct 28, 2014.

    1. exaccuss

      exaccuss Active Member

      Joined:
      Nov 10, 2013
      Messages:
      1,021
      Likes Received:
      6
      Trophy Points:
      38
      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?
       
    2. Endus

      Endus Community Developer

      Joined:
      Jul 9, 2012
      Messages:
      458
      Likes Received:
      6
      Trophy Points:
      18
      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));
       
      Last edited: Oct 28, 2014
    3. Funkwheat

      Funkwheat New Member

      Joined:
      Dec 12, 2013
      Messages:
      64
      Likes Received:
      0
      Trophy Points:
      0
    4. exaccuss

      exaccuss Active Member

      Joined:
      Nov 10, 2013
      Messages:
      1,021
      Likes Received:
      6
      Trophy Points:
      38
      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...
       
    5. Funkwheat

      Funkwheat New Member

      Joined:
      Dec 12, 2013
      Messages:
      64
      Likes Received:
      0
      Trophy Points:
      0
      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.
       
    6. exaccuss

      exaccuss Active Member

      Joined:
      Nov 10, 2013
      Messages:
      1,021
      Likes Received:
      6
      Trophy Points:
      38
      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.
       
    7. Endus

      Endus Community Developer

      Joined:
      Jul 9, 2012
      Messages:
      458
      Likes Received:
      6
      Trophy Points:
      18
      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.
       
    8. nekoramen

      nekoramen Member

      Joined:
      Dec 1, 2013
      Messages:
      38
      Likes Received:
      3
      Trophy Points:
      8
      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.
       
    9. exaccuss

      exaccuss Active Member

      Joined:
      Nov 10, 2013
      Messages:
      1,021
      Likes Received:
      6
      Trophy Points:
      38
      Thanks for that neko, it's good to see you back here again. Can't wait until he releases a fix.
       
    10. kaihaider

      kaihaider Community Developer

      Joined:
      May 18, 2010
      Messages:
      1,325
      Likes Received:
      5
      Trophy Points:
      38
      Can't you use a sequence instead :S
       
    11. nekoramen

      nekoramen Member

      Joined:
      Dec 1, 2013
      Messages:
      38
      Likes Received:
      3
      Trophy Points:
      8
      ...
      ...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
       
    12. Fog

      Fog New Member

      Joined:
      Oct 2, 2012
      Messages:
      25
      Likes Received:
      0
      Trophy Points:
      0
      What does sequence code look like?
       
    13. kaihaider

      kaihaider Community Developer

      Joined:
      May 18, 2010
      Messages:
      1,325
      Likes Received:
      5
      Trophy Points:
      38
      for trees: return new Sequence( //with Decorators, selectors or w/e

      for coroutines: await a && await b & await c
       
    14. Fog

      Fog New Member

      Joined:
      Oct 2, 2012
      Messages:
      25
      Likes Received:
      0
      Trophy Points:
      0
      I'm no coder so most of that looks foreign to me :). Any examples of how I could use that in the Magitek routine?
       
    15. kaihaider

      kaihaider Community Developer

      Joined:
      May 18, 2010
      Messages:
      1,325
      Likes Received:
      5
      Trophy Points:
      38
      I don't know magitek routine and looking at it would just slow down coding of my routine :p

      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.
       
    16. bbcd

      bbcd New Member

      Joined:
      Sep 20, 2014
      Messages:
      15
      Likes Received:
      0
      Trophy Points:
      0
      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
       
    17. Fog

      Fog New Member

      Joined:
      Oct 2, 2012
      Messages:
      25
      Likes Received:
      0
      Trophy Points:
      0
      It actually got fixed. LastSpell now recognizes "Spinning Edge"
       
    18. kaihaider

      kaihaider Community Developer

      Joined:
      May 18, 2010
      Messages:
      1,325
      Likes Received:
      5
      Trophy Points:
      38
      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
       
    19. Endus

      Endus Community Developer

      Joined:
      Jul 9, 2012
      Messages:
      458
      Likes Received:
      6
      Trophy Points:
      18
      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).
       
    20. exaccuss

      exaccuss Active Member

      Joined:
      Nov 10, 2013
      Messages:
      1,021
      Likes Received:
      6
      Trophy Points:
      38
      It looks like the problem is solved now, thank you all for your input and help :)
       

    Share This Page