Planeswalker Redirection

by **friarsol** » 14 Jul 2012, 16:22

Damn Hellfish. This is genius.

We should probably flesh out DealDamage AI to properly target Human when they actually want to target a PW.

We should be able to phase out the simplified Targeting {Tgt}[CP, P, C] now that we don't need it for including Planeswalkers.

by **moomarc** » 14 Jul 2012, 16:25

Just saw this as well. Great stuff! =D>

by **Hellfish** » 14 Jul 2012, 16:56

*phew* I thought you had found a problem already.

~~There is rudimentary AI in that if it can't put you in burn range (<5 I think I put) it will choose to redirect.~~ EDIT: Nvm, I think that check is ineffective, because of AF_DealDamage AI as Sol said.

by **friarsol** » 14 Jul 2012, 20:15

Hellfish wrote:~~There is rudimentary AI in that if it can't put you in burn range (<5 I think I put) it will choose to redirect.~~ EDIT: Nvm, I think that check is ineffective, because of AF_DealDamage AI as Sol said.

It might be functional enough for things like Pestilence, but with some added code in DealDamage the AI can start targeting the PWs with more focus.

by **Sloth** » 14 Jul 2012, 20:23

Hellfish wrote:*phew* I thought you had found a problem already.

~~There is rudimentary AI in that if it can't put you in burn range (<5 I think I put) it will choose to redirect.~~ EDIT: Nvm, I think that check is ineffective, because of AF_DealDamage AI as Sol said.

I agree with Sol. It's not really ineffective (it should work with Blightning, Anathemancer, Cinder Pyromancer, etc.). Since there hasn't been any code in DealDamageCanPlayAI that targets Planeswalkers this is a big improvement.