• Visit Rebornbuddy
  • Visit Panda Profiles
  • Visit LLamamMagic
  • Blood DK - Tank Edition

    Discussion in 'Archives' started by Pasterke, May 20, 2014.

    1. zjxlshb

      zjxlshb New Member

      Joined:
      Oct 17, 2014
      Messages:
      26
      Likes Received:
      0
      Trophy Points:
      0
      support plug in pull more??
       
    2. Jupken23

      Jupken23 New Member

      Joined:
      Oct 29, 2014
      Messages:
      27
      Likes Received:
      1
      Trophy Points:
      0
      Nice rotation! Would it be possible to add plague leech + glyphed outbreak? Other than that it does the job quite quite well!
       
    3. zazey

      zazey Member

      Joined:
      Jun 15, 2012
      Messages:
      71
      Likes Received:
      1
      Trophy Points:
      8
      Any way to add soul reaper sub 35?
       
    4. Pasterke

      Pasterke Well-Known Member Buddy Store Developer

      Joined:
      Dec 12, 2011
      Messages:
      1,228
      Likes Received:
      48
      Trophy Points:
      48
      Like the OP mentioned, this CR is for WoD 6.0.2. My DK is still lvl 90 and once I get there to lvl my DK up, I will write a new CR for the current content.

      It's complete free and I don't need to update it. In the 1st place, I try to enjoy the game. My main is a druid and I'm still tweaking my CR for the current content. When everything goes well with my Druid CR, then I will start update other CR's.

      I'm not a professional programmer and have no obligations for a free routine.

      I leveled my Paladin and Elemental shaman, and I have working CR's for those classes, but not optimal for endgame content.

      I'm busy, but when my DK is on turn, you may have a whole new routine, because I use the new style what HB will use as standard in the next future.

      Be patient and enjoy the game as I do.
       
    5. ZyTE

      ZyTE New Member

      Joined:
      Jan 15, 2010
      Messages:
      177
      Likes Received:
      0
      Trophy Points:
      0
      Thanks for even coding this CC, for a non developer this is amazing!
       
    6. Jupken23

      Jupken23 New Member

      Joined:
      Oct 29, 2014
      Messages:
      27
      Likes Received:
      1
      Trophy Points:
      0
      I've talked with Pasterke and will upload my files. I've made some modifications to his rotation in order to add Plague Leech with Glyphed Outbreak.
      It's not 100% optimal but does the job quite nicely. I will post this at a later time after doing a bit more testing.
       
    7. Jupken23

      Jupken23 New Member

      Joined:
      Oct 29, 2014
      Messages:
      27
      Likes Received:
      1
      Trophy Points:
      0
      View attachment DeathKnight.zip

      Changes
      - Implemented Plague Leech.
      - Focuses less on AoE and more on single target with Death Strike use.
      - Requires Outbreak to be glyphed.

      Please provide feedback.
       
      UREG likes this.
    8. Gamblor

      Gamblor New Member

      Joined:
      May 12, 2013
      Messages:
      17
      Likes Received:
      0
      Trophy Points:
      0
      The attached file doesn't work.
       
    9. Jupken23

      Jupken23 New Member

      Joined:
      Oct 29, 2014
      Messages:
      27
      Likes Received:
      1
      Trophy Points:
      0
      How exactly? Are you loading honorbuddy while being in blood spec with the entire map placed in "routines" folder?
      Are are you unable to download?
       
    10. UREG

      UREG New Member

      Joined:
      Mar 19, 2010
      Messages:
      101
      Likes Received:
      1
      Trophy Points:
      0
      Thanks for the update seems a lot smoother now with glyphed outbreak and plague leech and priority on death strike but I would like to have Defile down even on single target fights as much as possible because it reduces damage taken by 10%

      Other than that I saw increase in damage output and survivability for sure.
       
    11. Gamblor

      Gamblor New Member

      Joined:
      May 12, 2013
      Messages:
      17
      Likes Received:
      0
      Trophy Points:
      0
      Figured my problem out, had problems extracting the file. The shortest way to put this is "windows 8 sucks".

      Going to try this modified CR tonight, will report back.
       
    12. PrimeTime

      PrimeTime Member

      Joined:
      Oct 9, 2012
      Messages:
      44
      Likes Received:
      0
      Trophy Points:
      6
      with the changes u added to the CR the selfhealing is Shitty! Death Strike and Blood Boil barely gets used!.... Idd let Pasterke handle it when he have the time tbh. Not saying ur modification is not a good idea but for me its not working. As I said DS and BB is used verry seldom..
       
      Last edited: Dec 7, 2014
    13. Jupken23

      Jupken23 New Member

      Joined:
      Oct 29, 2014
      Messages:
      27
      Likes Received:
      1
      Trophy Points:
      0
      I'll try to see if I can make defile cast single target, I didn't look for that yet. So far defile is cast the moment there are 2 targets or more in range.

      Did you glyph outbreak? There should be an increase of Death Strike tbh. I also specifically mentioned that it requires glyphed outbreak. I also recommend the use of Plague Leech.
      Other than that I don't see what spells it casts, can you post the log of order of spells?
       
      Last edited: Dec 7, 2014
    14. robinekstrom1

      robinekstrom1 New Member

      Joined:
      Oct 18, 2012
      Messages:
      34
      Likes Received:
      0
      Trophy Points:
      0
      how do i install this i always had problem to install routines
       
    15. Fl3x1

      Fl3x1 New Member

      Joined:
      Jul 31, 2011
      Messages:
      27
      Likes Received:
      0
      Trophy Points:
      1
      Just unzip the DearthKnight folder in your honorbuddy Routines folder [Yourhonorbuddy\Routines\DeatKnight]
       
      Last edited: Dec 9, 2014
    16. hestee

      hestee New Member

      Joined:
      May 21, 2014
      Messages:
      3
      Likes Received:
      0
      Trophy Points:
      1
      Would be better to leave it as is, and then add a option to hold, or press a button, to make it cast on single target. Dont want it to waste it on some thrash mob, not to be up for a aoe.

      Also a implementation of dark sim, would be godly, but that requires a lot more work ;)

      Huge problem with Death Strike, dunno if you uploaded a wrong version, you are calling S.castDeathStrike0(), that contains so strange checks.

      && !U.debuffExists(FROST_FEVER, Me.CurrentTarget)
      && !U.debuffExists(BLOOD_PLAGUE, Me.CurrentTarget)
      && RPcount >= 10

      So will only cast if unit doesnt have diseases, shouldnt be too often for a DK ;)

      Removing the 0 seems to fix it, will test in a bit ;)

      Also noticed this in death strike:
      && (!U.spellOnCooldown(DEATH_STRIKE) || U.buffExists(CRIMSON_SCOURGE, Me))

      what does CRIMSON_SCOURGE have to do with death strike, seems like it will halt the routine quite a bit, if it tries to cast death strike on CRIMSON_SCOURGE procs :x
       
      Last edited: Dec 9, 2014
    17. Jupken23

      Jupken23 New Member

      Joined:
      Oct 29, 2014
      Messages:
      27
      Likes Received:
      1
      Trophy Points:
      0
      Was planning for this.

      Aint nobody got time for that. Don't even know what spells do work with it.

      Please try to follow my train of thought why I coded it like this.

      1. Blood Boil gives 20% more healing on the next Death Strike (this effect can even stack to 40% or more). I'm not lying, read the Blood Boil tooltip in game. I think the buff is called Scent of Blood.

      2. DeathStrikeO is a different variable name. I could even name it PandaIsAwesome for all I care, it's done on purpose. I intended to create a Death Strike different from the existing Death Strike that Pasterke already implemented.
      && !U.debuffExists(FROST_FEVER, Me.CurrentTarget)
      && !U.debuffExists(BLOOD_PLAGUE, Me.CurrentTarget)
      && RPcount >= 10
      The reason behind this code, this death strike, is to make it use Death Strike instead of recasting diseases manually.
      Death Strike gives 20RP (notice RPcount >= 10). Outbreak costs 30Rp while glyphed.
      If this would be removed I'd have Plague Leech > Icy Touch > Plague Strike > ...
      While now it does Plague Leech > Death Strike > Outbreak.

      The main idea of using Plague Leech is to get an extra Death Strike, not recasting diseases. This is why it was implemented in this way.

      But if you can do better, by all means show me!
       
    18. hestee

      hestee New Member

      Joined:
      May 21, 2014
      Messages:
      3
      Likes Received:
      0
      Trophy Points:
      1
      But still, you cant cast death strike if it's on cd, what this check does is Cast death strike if, it's not on cd OR if you have a free blood boil, you could do a && check for scent of blood, if you want higher prio on death strike with the buff up.
      You are casting death strike whenever you can anyway, so doesnt make sense to check if you have the buff.

      I'm not trying to pull you down, im trying to help improve it.
      Fixed blood tap btw, since it wasnt working properly. it didnt check if you had any free slots before casting it, so it halted routine, at the same time it was added to GCD, but not needed with blood tap.

      Code:
              public static Composite castBloodTap()
              {
                  return new Decorator(ret => gotTarget
                      && SpellManager.HasSpell(BLOOD_TAP)
                      && SpellManager.CanCast(BLOOD_TAP)
                      && Me.CurrentTarget.IsWithinMeleeRange
                      && (U.buffExists(BLOOD_CHARGE, Me)
                      && U.buffStackCount(BLOOD_CHARGE, Me) >= 5)
                      && ((Me.GetRuneCount(0) + Me.GetRuneCount(1) == 0) || (Me.GetRuneCount(2) + Me.GetRuneCount(3) == 0) || (Me.GetRuneCount(4) + Me.GetRuneCount(5) == 0)),
                      new Action(ret =>
                      {
                          SpellManager.Cast(BLOOD_TAP);
                          Logging.Write(Colors.Yellow, WoWSpell.FromId(BLOOD_TAP).Name);
                          LastSpell = BLOOD_TAP;
                      }));
              }
      
      Hope it was more usefull than my last post.
       
    19. Jupken23

      Jupken23 New Member

      Joined:
      Oct 29, 2014
      Messages:
      27
      Likes Received:
      1
      Trophy Points:
      0
      It checks for free Blood Boils. If Blood Boil is cast before Death Strike, it increases it's healing. At the same time there another mechanic in place.
      LastSpell = DEATH_STRIKE;
      SetNextSpellAllowed();
      Basically the concept that OP had in mind was to never allow 2x same spell being cast consequently. It won't cast 2x Death Strike unless there's no other option.
      Also, the main reason that DeathStrikeO is there to cast Death Strike without having diseases up, right after the use of Plague Leech.
      Remove this and you'll find yourself re-applying diseases rather than casting Death Strike while you'd have a glyphed outbreak, it's a waste.
      The buff check is something added from OP's original Death Strike. Since the rotation goes as I want it, I don't feel the need to remove it.
      If there is a more simple solution to this, please tell me.
      I appreciate feedback, don't get me wrong. Other than that I don't use Blood Tap atm, ty for pointing it out.
      I will change this whenever I feel like it (don't have much time on my hands this week).
       
    20. zazey

      zazey Member

      Joined:
      Jun 15, 2012
      Messages:
      71
      Likes Received:
      1
      Trophy Points:
      8
      The Zip file you uploaded Jupken is an invalid file. Tried with multiple zip programs on 2 different computers, any way you can reupload?
       

    Share This Page