forge icon indicating copy to clipboard operation
forge copied to clipboard

AI casts Scale Up only for overload cost

Open MorihiroS opened this issue 2 years ago • 5 comments

Describe the bug AI doesn't cast Scale Up even if it has untapped Forest. It casts it only for overload cost

To Reproduce Steps to reproduce the behavior:

  1. AI has a 1/1 creature and untapped Forest.
  2. AI has Scale Up in its hands
  3. AI doesn't cast Scale Up

Expected behavior AI should cast Scale Up for 1 creature and it should cast it for overload cost with 2+ creatures

Desktop OS: Windows 10 Java Version: 1.8.0_202-b08 Forge Version: 1.6.60-SNAPSHOT-12.08

Screenshots

scale

MorihiroS avatar Dec 09 '23 21:12 MorihiroS

@Hanmac, need AI label, keep label and assign Agetian, please

MorihiroS avatar Jan 08 '24 07:01 MorihiroS

AI casts it, but only for overload cost

scale1

MorihiroS avatar Jul 17 '24 21:07 MorihiroS

@Agetian, I think it's easy to fix, right? Or am I wrong?

MorihiroS avatar Aug 29 '24 05:08 MorihiroS

@Agetian @tool4ever can we add something like "OverloadNeedsToPlay" SVar to better optimize it?

Or can we add better isOverload check in the SpellEffectAi parts, to not use Overload when there is <1 that would affect it?

Hanmac avatar Aug 29 '24 07:08 Hanmac

I think the isOverload approach would be cleaner if it's more or less universal. If we need additional differentiation on per-card basis, then we can add OverloadNeedsToPlay, yeah.

Agetian avatar Aug 29 '24 10:08 Agetian