Page 71 of 86

Re: SVN Bug Reports

PostPosted: 12 Jun 2011, 20:33
by friarsol
lazylockie wrote:r9622

Allies are still triggering twice when themselves enter the battlefield (since revision 9000 I think). So a lone Hada Freeblade starts at 2/3, instead of 1/2.
I just tested this and did not have this problem. Are you sure your card file is up to date?

Re: SVN Bug Reports

PostPosted: 12 Jun 2011, 20:34
by Sloth
Jaedayr wrote:r9621

AI blocked Norwood Riders with Old Ghastbark and Safehold Duo.
I was sure this wasn't possible anymore. :(
I put in some safety measures. Shouldn't happen anymore. Thanks Jaedayr.

Re: SVN Bug Reports

PostPosted: 12 Jun 2011, 21:01
by lazylockie
friarsol wrote:
lazylockie wrote:r9622

Allies are still triggering twice when themselves enter the battlefield (since revision 9000 I think). So a lone Hada Freeblade starts at 2/3, instead of 1/2.
I just tested this and did not have this problem. Are you sure your card file is up to date?
I just checked the codes and they are the same. It doesn't make sense that mine triggers twice and yours don't, because from the code I think it would indeed trigger twice:

Name:Hada Freeblade
ManaCost:W
Types:Creature Human Soldier Ally
Text:no text
PT:0/1
T:Mode$ ChangesZone | Origin$ Any | Destination$ Battlefield | ValidCard$ Card.Self | OptionalDecider$ You | Execute$ TrigPutCounter | TriggerDescription$ Whenever CARDNAME or another Ally enters the battlefield under your control, you may put a +1/+1 counter on CARDNAME.
T:Mode$ ChangesZone | Origin$ Any | Destination$ Battlefield | ValidCard$ Ally.YouCtrl | TriggerZones$ Battlefield | OptionalDecider$ You | Execute$ TrigPutCounter | Secondary$ True | TriggerDescription$ Whenever CARDNAME or another Ally enters the battlefield under your control, you may put a +1/+1 counter on CARDNAME.
SVar:TrigPutCounter:DB$PutCounter | Defined$ Self | CounterType$ P1P1 | CounterNum$ 1
SVar:PlayMain1:TRUE
SVar:BuffedBy:Ally
SVar:RemRandomDeck:True
SVar:Rarity:Uncommon
SVar:Picture:http://www.wizards.com/global/images/magic/general/hada_freeblade.jpg
SetInfo:WWK|Uncommon|http://magiccards.info/scans/en/wwk/7.jpg
End
Isn't it supposed to be Ally.Other+YouCtrl?

Re: SVN Bug Reports

PostPosted: 12 Jun 2011, 21:11
by friarsol
lazylockie wrote:Isn't it supposed to be Ally.Other+YouCtrl?
How are your Allies coming into play? Are you casting them from your hand? Is more than one entering play at the same time? Does your issue go away if you add that line?

If your cardfile is the same, maybe you changed something in the code to cause the double trigger?

Re: SVN Bug Reports

PostPosted: 12 Jun 2011, 22:51
by lazylockie
friarsol wrote:
lazylockie wrote:Isn't it supposed to be Ally.Other+YouCtrl?
How are your Allies coming into play? Are you casting them from your hand? Is more than one entering play at the same time? Does your issue go away if you add that line?

If your cardfile is the same, maybe you changed something in the code to cause the double trigger?
I've tested a bit more, the issue is with AI's allies.

Code: Select all
AICardsInHand=Hada Freeblade;Kazandu Blademaster;Mox Pearl;Mox Pearl;Mox Pearl
Changing to Ally.Other+YouCtrl does solve the problem for AI.

Re: SVN Bug Reports

PostPosted: 13 Jun 2011, 08:50
by Sloth
lazylockie wrote:
friarsol wrote:
lazylockie wrote:Isn't it supposed to be Ally.Other+YouCtrl?
How are your Allies coming into play? Are you casting them from your hand? Is more than one entering play at the same time? Does your issue go away if you add that line?

If your cardfile is the same, maybe you changed something in the code to cause the double trigger?
I've tested a bit more, the issue is with AI's allies.

Code: Select all
AICardsInHand=Hada Freeblade;Kazandu Blademaster;Mox Pearl;Mox Pearl;Mox Pearl
Changing to Ally.Other+YouCtrl does solve the problem for AI.
I fixed this for all allies. But I find it rather alarming that we have triggers that work differently for the AI.

Re: SVN Bug Reports

PostPosted: 13 Jun 2011, 13:28
by slapshot5
r9613

Is there a known issue with AI playing abilities with snow mana in the Cost?

I was testing Adarkar Windform which has an activation cost of "1 S", and the computer just would not play this ability. It wasn't even checking canPlayAI in my AF!

I changed the cost to "1 U" for testing and the computer played it right away. canPlayAI was now getting checked.

Are cards like this just supposed to have RemAIDeck set, or do we have a bug here?

Thanks,
slapshot5

Re: SVN Bug Reports

PostPosted: 13 Jun 2011, 13:33
by Chris H.
slapshot5 wrote:r9613

Is there a known issue with AI playing abilities with snow mana in the Cost?

I was testing Adarkar Windform which has an activation cost of "1 S", and the computer just would not play this ability. It wasn't even checking canPlayAI in my AF!

I changed the cost to "1 U" for testing and the computer played it right away. canPlayAI was now getting checked.

Are cards like this just supposed to have RemAIDeck set, or do we have a bug here?
`
I do not think that the AI can handle "S" costs at this time.

Sloth created a snow AI deck last your for quest mode and upon testing we discovered this issue.

And yes, give the card a RemAIDeck SVar.

Re: SVN Bug Reports

PostPosted: 13 Jun 2011, 14:23
by Agetian
I noticed that Runeboggle does not work as expected (r9652): I was casting Viashino Fangtail for all my mana, the AI cast Runeboggle. I was given a dialog to pay 1, which I couldn't do, so I clicked Cancel, but Visahino Fangtail still came into play instead of going into my graveyard. A test scenario:

runeboggle_test.txt:

Code: Select all
HumanCardsInPlay=Mountain; Mountain; Mountain; Forest
AICardsInPlay=Plains; Island; Swamp; Swamp
HumanCardsInHand=Viashino Fangtail
AICardsInHand=Runeboggle
ActivePlayer=Human
ActivePhase=Main1

Re: SVN Bug Reports

PostPosted: 13 Jun 2011, 15:32
by UnderFlow
R9657:
The first triggered ability of Vorinclex, Voice of Hunger uses the stack, even when it is triggered by a mana ability.
605.1b. A triggered ability without a target that triggers from activating a mana ability and could put mana into a player's mana pool when it resolves is a mana ability.
Gauntlet of Might has the same issue.

Re: SVN Bug Reports

PostPosted: 13 Jun 2011, 19:42
by Sloth
UnderFlow wrote:R9657:
The first triggered ability of Vorinclex, Voice of Hunger uses the stack, even when it is triggered by a mana ability.
605.1b. A triggered ability without a target that triggers from activating a mana ability and could put mana into a player's mana pool when it resolves is a mana ability.
Gauntlet of Might has the same issue.
Good point. We should fix this. Might be a lot of work though.

Re: SVN Bug Reports

PostPosted: 13 Jun 2011, 21:17
by pwangsta
r9670:

Chromatic Sphere - got mana, did not draw card
Eternal Flame - opponent only took 1/2 damage, not full
Arc Trail - allowed me to do 3 damage to a single creature or a single player
Creatures with islandwalk were unblockable in one game even when i didn't control an island (and wasn't playing blue at all)
Kazandu Blademaster - ETB ability (+1/+1 counter) triggers twice
Suffer the Past - requires 2UB to cast, where it should be xB
When AI played Amnesia, the following exception was thrown:
| Open
An error has occurred. You can copy/paste this message or save it to a file.
Please report this, plus what you tried to do, to:
viewforum.php?f=26
If you don't want to register an account, you can mail it directly to
mtgerror@yahoo.com


There is an error in the card code for Amnesia:



Version:
Forge -- official beta: $Date: 2011-01-06 11:34:48 -0500 (Thu, 06 Jan 2011) $, SVN revision: $Revision: 4891 $

OS: Windows 7 Version: 6.1 Architecture: x86

Java Version: 1.6.0_21 Vendor: Sun Microsystems Inc.

Detailed error trace:
java.lang.NullPointerException
at forge.card.abilityFactory.AbilityFactory_ZoneAffecting.discardCanPlayAI(Unknown Source)
at forge.card.abilityFactory.AbilityFactory_ZoneAffecting.access$1(Unknown Source)
at forge.card.abilityFactory.AbilityFactory_ZoneAffecting$8.canPlayAI(Unknown Source)
at forge.ComputerAI_General.getPlayable(Unknown Source)
at forge.ComputerAI_General.getMain1(Unknown Source)
at forge.ComputerAI_General.playCards(Unknown Source)
at forge.ComputerAI_General.main1(Unknown Source)
at forge.ComputerAI_Input.think(Unknown Source)
at forge.ComputerAI_Input.showMessage(Unknown Source)
at forge.GuiInput.setInput(Unknown Source)
at forge.GuiInput.update(Unknown Source)
at java.util.Observable.notifyObservers(Unknown Source)
at java.util.Observable.notifyObservers(Unknown Source)
at forge.MyObservable.updateObservers(Unknown Source)
at forge.Phase.nextPhase(Unknown Source)
at forge.GuiDisplay4$27.actionPerformed(Unknown Source)
at javax.swing.AbstractButton.fireActionPerformed(Unknown Source)
at javax.swing.AbstractButton$Handler.actionPerformed(Unknown Source)
at javax.swing.DefaultButtonModel.fireActionPerformed(Unknown Source)
at javax.swing.DefaultButtonModel.setPressed(Unknown Source)
at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(Unknown Source)
at java.awt.Component.processMouseEvent(Unknown Source)
at javax.swing.JComponent.processMouseEvent(Unknown Source)
at java.awt.Component.processEvent(Unknown Source)
at java.awt.Container.processEvent(Unknown Source)
at java.awt.Component.dispatchEventImpl(Unknown Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(Unknown Source)
at java.awt.LightweightDispatcher.retargetMouseEvent(Unknown Source)
at java.awt.LightweightDispatcher.processMouseEvent(Unknown Source)
at java.awt.LightweightDispatcher.dispatchEvent(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.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)

Re: SVN Bug Reports

PostPosted: 13 Jun 2011, 22:00
by slapshot5
pwangsta wrote:r9670:
Suffer the Past - requires 2UB to cast, where it should be xB
When AI played Amnesia, the following exception was thrown:
These two issues are now fixed in SVN.

-slapshot5

Re: SVN Bug Reports

PostPosted: 13 Jun 2011, 23:13
by Chris H.
pwangsta wrote:Eternal Flame - opponent only took 1/2 damage, not full
`
Fixed, thank you pwangsta.

Re: SVN Bug Reports

PostPosted: 14 Jun 2011, 00:32
by Chris H.
pwangsta wrote:r9670:

Chromatic Sphere - got mana, did not draw card
`
Using rev9678 my Chromatic Sphere gave me mana and I drew a card.