forge icon indicating copy to clipboard operation
forge copied to clipboard

AI activates some activated abilities as soon as it can with no reason

Open MorihiroS opened this issue 1 year ago • 9 comments

Describe the bug AI activates activated abilities of some cards as soon as it can with no reason. It's ineffectively. Some of this cards:

  • [x] Ugin's Labyrinth (second ability)
  • [x] Canopy lands
  • [x] Zuran Orb
  • [ ] Skirk Prospector
  • [x] Eladamri, Korvecdal
  • [x] Pyrite Spellbomb

To Reproduce Steps to reproduce the behavior:

  1. AI plays Ugin's Labyrinth
  2. AI uses second ability of Ugin's Labyrinth immediately

or

  1. AI play Fiery Islet and casts Ragavan, Nimble Pilferer
  2. AI successfully attacks with Ragavan, Nimble Pilferer and takes Treasure token
  3. AI sacrifices Fiery Islet and now it hasn't lands

or

  1. AI casts Zuran Orb
  2. AI sacrifices all of its lands immediately

or

  1. AI has one untapped land
  2. AI casts Skirk Prospector for land's mana
  3. AI sacrifices Skirk Prospector and casts Aether Vial

or

  1. AI has Eladamri, Korvecdal and 2 other little creatures
  2. AI doesn't have big creatures in its hands and on top of its library
  3. AI activates Eladamri, Korvecdal for nothing

or

  1. AI has two untapped lands
  2. AI casts Pyrite Spellbomb
  3. AI sacrifices Pyrite Spellbomb and draw a card

Expected behavior AI should activate activated abilities of this and similar cards in accordance with the situation. For example:

  • AI activates second ability of the Ugin's Labyrinth only after it has enough mana to cast exiled card or in response to an attempt to destroy the Ugin's Labyrinth;
  • AI sacrifices canopy lands only after it already has X lands and/or no cards in its hand
  • I don't know, may be AI should activate the Zuran Orb ability when it has less than X life or when Titania, Protector of Argoth is on the battlefield
  • AI doesn't sacrifice Skirk Prospector before it has 2+ lands and only for cast a spell with cmc = number of its lands + 1
  • AI should activate Eladamri, Korvecdal if it has big creatures in its hands and on top of its library
  • AI should sacrifices Pyrite Spellbomb and draw a card only if it hasn't cards in hands. Otherwise, AI should sacrifices Pyrite Spellbomb to deal fatal damage.

Additional context Similar with #4214

Desktop

  • OS: Windows 10
  • Java Version: 1.8.0_202-b08
  • Forge Version: 1.6.64-SNAPSHOT-07-13

Screenshots

labir canopy canopy2 zuran skirk elad1 elad2 elad3 pyri

MorihiroS avatar Jul 14 '24 20:07 MorihiroS

Added a problem with Eladamri, Korvecdal

MorihiroS avatar Aug 10 '24 14:08 MorihiroS

@Hanmac, need AI label, keep label and assign someone, please, Issue contains 2 MH3 cards!

MorihiroS avatar Aug 12 '24 19:08 MorihiroS

@Agetian, is it realistic to fix at least the first two cards? The first one prevents the AI from playing a deck with eldrazi, the second one breaks the manabase of fast aggro decks

MorihiroS avatar Aug 27 '24 04:08 MorihiroS

Can you please clarify which two cards specifically you're referring to? I can take a look, though I can't promise to do it very quickly, sadly.

Agetian avatar Aug 27 '24 10:08 Agetian

@Agetian, no problem! The first two on the list: Ugin's Labyrinth (second ability) and Canopy lands

MorihiroS avatar Aug 27 '24 19:08 MorihiroS

Add Pyrite Spellbomb

MorihiroS avatar Sep 08 '24 17:09 MorihiroS

This seems related, from DSK: Marvin, Murderous Mimic immediately becomes +3/+3 on enter because Possessed Goat has a: Pay 3, Discard a card activated ability. It activates abilities of other creates immediately without paying activation costs.

If this is unrelated please let me know I can start a new Issue.

@richardleggett, if it activates abilities of other creatures immediately without paying activation costs, this requires a separate issue

MorihiroS avatar Sep 17 '24 20:09 MorihiroS

@Agetian, can you do something with at least Ugin's Labyrinth (second ability) and Canopy lands? It's unplayable... Actual for 2.0.00-SNAPSHOT-10.23

burn1 ugin1 ugin2

MorihiroS avatar Oct 23 '24 08:10 MorihiroS

@Agetian, I'm pleasantly bummed, wasn't hoping for those annoying bugs to be fixed anymore! Thank you!

There is the same problem with Greater Gargadon, but I'm not sure how to find the optimal condition for AI use

MorihiroS avatar Oct 26 '24 21:10 MorihiroS

@Agetian, Add Greater Gargadon

MorihiroS avatar Feb 12 '25 11:02 MorihiroS

@MorihiroS Greater Gargadon is part of a Combo with Balance Effects I don't think there is a nice logic for this to be used (instead of never)

Hanmac avatar Feb 12 '25 11:02 Hanmac

@Hanmac I already wrote about Restore Balance in Expected behavior, but I'll add Balance, Balancing Act and Magus of the Balance

And I have already described in the same place how it is approximately possible to realize it

MorihiroS avatar Feb 28 '25 01:02 MorihiroS

Add Warden of the Inner Sky

MorihiroS avatar May 20 '25 12:05 MorihiroS