[Release] Healing (Manual Targeting)

[SirSik]

it is definitely "Force-imbalance". But when I try to run it, this error:

2017-04-20 15:00:39,624 [7] ERROR AssemblyLoader`1 - Compiler Error: d:\SWTOR xxx\Routines\DefaultCombat\Helpers\Logger.cs(23,4) : warning CS0618: 'Buddy.Common.Logging.Write(System.Windows.Media.Color, string, params object[])' is obsolete: 'Logging.WriteX is no longer used, declare [private ILog _log = Log.Get();] in your type and use its members instead!'
2017-04-20 15:00:39,625 [7] ERROR AssemblyLoader`1 - Compiler Error: d:\SWTOR xxx\Routines\DefaultCombat\Routines\Advanced\Sage\seer.cs(89,6) : error CS1501: No overload for method 'HoT' takes 5 arguments

So, something is not happy

Spoiler: Click here to see the fix

sorry, i shouldn't be doing these releases without testing them....
So to give you some more information: The logger.cs pretty much always comes up, it's just a warning.
The Seer one has come up as for when it looks to "Spell.cs" on what to do when it's being told to cast Spell.Hot, is it has 5 arguments. When i just checked, next to force armour it states:

Spell.HoT("Force Armor", 100, on => Me, 100, ret => !Me.CurrentTarget.HasDebuff("Force-imbalance")),

As you can see, I've F***ed up and put in 6 arguments. That is trying to cast, force armour, if the target has 100% hp, on me, if the target has 100% hp, and as long as my current target doesn't have the debuff "Force-imbalance" (yes i've just realised i messed up twice)

So mistake number 1. It's checking for HP% twice... remove the first instance of "100,"
and the mistake i realised when i was writing this - It's looking to my TARGET for the debuff of force-imbalance when it should be looking to see if the debuff is on me.

so removing the first mistake and fixing the other should make the working version look like this (i've also added an extra check into it which checks to make sure you're in combat, that way it doesn't keep bubbling you when you;re out of combat)

Spell.HoT("Force Armor", on => Me, 100, ret => Me.InCombat && !Me.HasDebuff("Force-imbalance")),

hope this gives you a better understanding of it all and i've (once again) updated the seer routine in this thread.

Updated Seer routine

 

[Praline]

Thank you so much for sharing your know how. I am enjoying learning with you.

 

[SirSik]

that's ok i know more than one person wanted this fixing and i know it's not the simplest, cleanest editing etc etc... it works for now
I'm back at work now for 4 days then on my week off after that i am going to try look for other ways to fix it with a fresh defaultcombat will see if i can actually get a new way for working out health percentage, i just need to learn to understand that api documentation that we have etc

 

[SirSik]

OK, Holy shit... I've fixed it already........

Ok... So, Praline, test this out for me.

Reinstall the ORIGINAL defaultcombat in the routines folder.
go into the Core folder and replace the text in targeting.cs with this and try that out. If you prefer that way to the manual targeting then i will continue to make improvements based on that method and redo this whole thread based on that.

 

[Praline]

Holy moly! I am speechless!
So Aevitas changes based on Pindleskin's suggestions did work. The only missing piece in the puzzle have been YOU all along to put it all together.
It worked well. In PVE it was as good as it was in the old days, or maybe better. It was healing my comp when needed, and zapping mobs like a pro. Do you think it will help to change healthpercent to stathealth for the cleansing bit to work as well? That would be a dream come true to be able to use this in OPS again. OMG you sir need a MEDAL!!!!

 

[kurfer]

So I added the spell.cs data in your original post and my operative still spams heals constantly. It's very random on who it selects; it's like an idiot is playing it. Refuses to self heal if I target myself and random will spam rotation on dudes with 100% health. It works semi OK with your auto file.

Am I to understand I keep everything original except for spell.cs with the manual heal?

 

[kurfer]

So it looks like this manual thing just spams abilities regardless of "health" in the order of your rotation and ignores all buff counts and other stuff.

Yeah this is how you get kicked right the F out of your guild for being a trash tier healer! No thanks!

 

[Praline]

So it looks like this manual thing just spams abilities regardless of "health" in the order of your rotation and ignores all buff counts and other stuff.

Yeah this is how you get kicked right the F out of your guild for being a trash tier healer! No thanks!

You can't read very well or follow instructions that was written very clearly, can you? Maybe best to delete your swtor folder and move on.

 

[SirSik]

Thing is the Med Op routine is written quite differently to Corruption for example.
The changes i made in spell.cs just turn off auto targeting...
but when you have something in a rotation that looks like this:

Spell.Cast("*Spell goes here*, ret => HealTarget.*blah blah*),

It will still cast on HealTarget which it gets from Targeting.
if you changed the spells to this then it will be ok

Spell.Cast("*Spell goes here*, ret => Me.CurrentTarget),

 

[kurfer]

You can't read very well or follow instructions that was written very clearly, can you? Maybe best to delete your swtor folder and move on.

I followed the instructions very clearly and have contributed with my limited knowledge of programming to have my changes to marauder routines make it in via an official update. Please don't white knight, white knights are very annoying.

What I am saying still stands, worthy attempt but IF a spell is constantly spammed regardless of health values it's not going to be efficient. The instructions for the manual targeting state to replace the spell.cs only. In addition he provides sorc/sage healing rotations. I tried those on my respective healers and everything performs the same way. Randomly targets myself (no tank target lines present) and will just spam the rotation in an odd order regardless of health on my target.

IF there is another step, it's not defined in the original thread.

I understand how the rotations work, my current Marauder tweaks are beating top tier players in ranked warzones atm. It outperforms 3k+ rated players currently. I have not modified anything else that would warrant this odd behavior so I can only assume this is working as intended.

I'm not worried about the health values as much as I am the additional variables in the equation.

With the sorc/sage what I *THINK* is causing it to constantly target myself is that there is code that ensures I always have a bubble on myself. Part of that code also checks to ensure I don't have the "debuff" associated when the bubble wears off. IF my bubble drops, it tries to target myself and reapply but can't so it's constantly re-targetting myself as I try to target a player who's health is dropping ultimately letting them die.

In the instance of the OP, you kinda need it to look at health. You want to maintain kolto probe on targets; you need the rotation to ensure you're maintaining 2 stacks of probe and reapplying 3-6 seconds before it drops. What I experience with the manual rotation fix is that it will spam probe over and over thus overlapping and wasting energy and not casting bigger heals when needed.