Page 1 of 1

Planeswalker Redirection

PostPosted: 14 Jul 2012, 16:22
by friarsol
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.

Re: Planeswalker Redirection

PostPosted: 14 Jul 2012, 16:25
by moomarc
Just saw this as well. Great stuff! =D>

Re: Planeswalker Redirection

PostPosted: 14 Jul 2012, 16:56
by Hellfish
*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.

Re: Planeswalker Redirection

PostPosted: 14 Jul 2012, 20:15
by friarsol
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.

Re: Planeswalker Redirection

PostPosted: 14 Jul 2012, 20:23
by Sloth
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.