Collectible Card Game Headquarters

Hanmac wrote:hm i need to test this more, (you might try to test it too)

i did try to get him to use more modes for Collective Brutality, but AI only ever uses the -2/-2 one. (even if it has a DiscardMe card for the Lifedrain choice)

and it would be cool if we can get the AI using Borrowed Grace right too.

===
i think the best idea would be for CharmAI, that when a spell has Escalate, it should try to use most modes if able.

It is tricky to play with optional choices since the AI will usually choose too many choices and be unable to pay, or waste resources paying for unnecessary options (eg. gain life instead of playing some other spell from hand). It had the same issue with strive from Theros block where it does not understand to stay within budget when adding targets. The concept of trading resources for versatility is lost on the AI because it has no overall strategy to begin with.

The evaluation of simultaneous effects is going to be even harder. The classic example is Kolaghan's Command where the AI will use two options on a X/2 artifact creature. The only reasonable solution I can think of is the brute force approach of trying all combinations and then evaluating the resultant state (ie. state rewinding) which is super-expensive and prone to bugs.

Hello, in r-31824, I enchanted Kor Spiritdancer with Debilitating Injury and Spiritdancer is still OTB

fmartel wrote:Hello, in r-31824, I enchanted Kor Spiritdancer with Debilitating Injury and Spiritdancer is still OTB

Did you read Kor Spiritdancer? It gets +2/+2 for each Aura attached to it. You attached it with an Aura.

that's what happen when you didn't take your morning coffee xD

Description: [In a commander game, game stuck at EOT "wating for opponent"]

ConcurrentModificationException | Open

Code: Select all

Forge Version:    1.5.55-SNAPSHOT-r31824
Operating System: Windows 7 6.1 amd64
Java Version:     1.8.0_101 Oracle Corporation

java.util.ConcurrentModificationException
   at java.util.LinkedList$ListItr.checkForComodification(Unknown Source)
   at java.util.LinkedList$ListItr.next(Unknown Source)
   at com.google.common.collect.Iterators$7.computeNext(Iterators.java:646)
   at com.google.common.collect.AbstractIterator.tryToComputeNext(AbstractIterator.java:143)
   at com.google.common.collect.AbstractIterator.hasNext(AbstractIterator.java:138)
   at forge.util.collect.FCollection.<init>(FCollection.java:90)
   at forge.game.card.CardCollection.<init>(CardCollection.java:115)
   at forge.game.card.CardLists.filter(CardLists.java:255)
   at forge.game.zone.PlayerZone.getCardsPlayerCanActivate(PlayerZone.java:117)
   at forge.game.player.Player.getCardsActivableInExternalZones(Player.java:1383)
   at forge.game.player.Player.getCardsIn(Player.java:1334)
   at forge.game.player.Player.getCardsIn(Player.java:1317)
   at forge.game.GameAction.checkStateEffects(GameAction.java:796)
   at forge.game.phase.PhaseHandler.startFirstTurn(PhaseHandler.java:901)
   at forge.game.GameAction.startGame(GameAction.java:1460)
   at forge.game.Match.startGame(Match.java:95)
   at forge.match.HostedMatch$2.run(HostedMatch.java:220)
   at java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
   at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
   at java.lang.Thread.run(Unknown Source)


:r31828
Chainer, Dementia Master has bug.
Creatures reanimated by Chainer should become black, not add to original color.

r31827

I had fevered visions out, at CPU's end step he had two cards in hand, fevered visions triggered, cpu drew a card (to a total of 3 cards in hand) but then CPU took two damage from Fevered Visions despite not having four cards in hand.

tjtillman wrote:r31827

I had fevered visions out, at CPU's end step he had two cards in hand, fevered visions triggered, cpu drew a card (to a total of 3 cards in hand) but then CPU took two damage from Fevered Visions despite not having four cards in hand.

Fixed (r31830).

- Agetian

Description: just opening forge

NullPointerException | Open

Code: Select all

Forge Version:    1.5.55-SNAPSHOT-r31835
Operating System: Windows 10 10.0 amd64
Java Version:     1.8.0_101 Oracle Corporation

java.lang.NullPointerException
   at forge.control.FControl.dispatchKeyEvent(FControl.java:392)
   at java.awt.DefaultKeyboardFocusManager.preDispatchKeyEvent(Unknown Source)
   at java.awt.DefaultKeyboardFocusManager.typeAheadAssertions(Unknown Source)
   at java.awt.DefaultKeyboardFocusManager.dispatchEvent(Unknown Source)
   at java.awt.Component.dispatchEventImpl(Unknown Source)
   at java.awt.Container.dispatchEventImpl(Unknown Source)
   at java.awt.Window.dispatchEventImpl(Unknown Source)
   at java.awt.Component.dispatchEvent(Unknown Source)
   at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
   at java.awt.EventQueue.access$500(Unknown Source)
   at java.awt.EventQueue$3.run(Unknown Source)
   at java.awt.EventQueue$3.run(Unknown Source)
   at java.security.AccessController.doPrivileged(Native Method)
   at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
   at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
   at java.awt.EventQueue$4.run(Unknown Source)
   at java.awt.EventQueue$4.run(Unknown Source)
   at java.security.AccessController.doPrivileged(Native Method)
   at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
   at java.awt.EventQueue.dispatchEvent(Unknown Source)
   at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
   at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
   at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
   at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
   at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
   at java.awt.EventDispatchThread.run(Unknown Source)


r31828

CPU had Furyblade Vampire on the board. At start of combat, it triggered. CPU discarded Insatiable Gorgers, Madness trigger for Insatiable Gorgers went onto the stack and the card went to exile; CPU only had 3 mana so could not pay the 4 to cast it. However, rather than go to the Graveyard, Insatiable Gorgers remained in exile for the rest of the game.

r31835

Slayer's Cleaver incorrectly gives +2/+2 to equipped creature instead of +3/+1

tjtillman wrote:r31835
Slayer's Cleaver incorrectly gives +2/+2 to equipped creature instead of +3/+1

Fixed (r31835).

- Agetian

tjtillman wrote:r31828

CPU had Furyblade Vampire on the board. At start of combat, it triggered. CPU discarded Insatiable Gorgers, Madness trigger for Insatiable Gorgers went onto the stack and the card went to exile; CPU only had 3 mana so could not pay the 4 to cast it. However, rather than go to the Graveyard, Insatiable Gorgers remained in exile for the rest of the game.

This happens on more than just this card, it happens quite frequently when the CPU discards a madness card and doesn't have the mana to cast it. Is this a known issue?

@tjtillman: yeah its a known issue.

friarsol wrote:I was thinking this might be because the AI tries to start casting the Madness spell and fails.

Generally how it should work:

1) Discard a madness card straight to exile
2) Choose whether you want to cast it or put it in your graveyard
2a) Cast
2b) Put in graveyard.

But the AI does

1) Discard a madness card straight to exile
2) Choose whether you want to cast it or put it in your graveyard
2a) Cast
2b) Put in graveyard.
3a) Fails to cast, (Return card to the zone it came from - Exile)

we just didn't find a way to fix it yet.
(i was looking at this moment, but i didn't find the place where it go wrong)

Hanmac wrote:we just didn't find a way to fix it yet.
(i was looking at this moment, but i didn't find the place where it go wrong)

I have an idea, I'll take a look this afternoon.