Page 3 of 9

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 29 Sep 2016, 04:40
by Agetian
l33t3rth4nur1111 wrote:Got this after using Toxic Deluge to (mostly) wipe the board, including an opponent's Sisters of Stone Death:

| Open
Game-15 > java.lang.RuntimeException: AbilityFactory : getAbility -- Sisters of Stone Death has no SVar: TrigForget
at forge.game.ability.AbilityFactory.getAbility(AbilityFactory.java:115)
at forge.game.trigger.TriggerHandler.runSingleTrigger(TriggerHandler.java:541)
at forge.game.trigger.TriggerHandler.runWaitingTrigger(TriggerHandler.java:337)
at forge.game.trigger.TriggerHandler.runWaitingTriggers(TriggerHandler.java:306)
at forge.game.GameAction.checkStateEffects(GameAction.java:872)
at forge.game.phase.PhaseHandler.startFirstTurn(PhaseHandler.java:905)
at forge.game.GameAction.startGame(GameAction.java:1462)
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)


Sure enough, sisters_of_stone_death.txt has

Code: Select all
SVar:DBForget:DB$ Pump | ForgetImprinted$ TriggeredCard
T:Mode$ ChangesZone | ValidCard$ Card.Self | Origin$ Battlefield | Destination$ Any | Execute$ TrigForget | Static$ True | Secondary$ True | TriggerDescription$ Forget all remembered cards.
SVar:DBCleanup:DB$ Cleanup | ClearRemembered$ True
but no TrigForget exists. There is a DBForget; maybe that was intended? Looking at the file history, it looks like leftovers during some code cleanup.
I'm pretty sure that DBCleanup was meant there. I'd like someone to take a look at the implemented fix though (r32231) to confirm.

- Agetian

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 29 Sep 2016, 06:07
by samotnjak23
Can you play online in forge, and if so what is the server name??

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 29 Sep 2016, 15:33
by cc-drake
I control several creatures with some +1/+1 counters, but when I cast Hazardous Conditions, all my creatures get -2/-2 :-(

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 29 Sep 2016, 17:51
by lujo
Merseine always asks for 1 colorless instead of the enchanted creature's converted mana cost.

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 29 Sep 2016, 18:41
by Agetian
lujo wrote:Merseine always asks for 1 colorless instead of the enchanted creature's converted mana cost.
I can confirm this but I'm not sure how to fix it. The card seems to use a really unique expression <1\EnchantedCost> to refer to the enchanted creature's mana cost, but that doesn't seem to work (was it not coded?) and I don't know what kind of alternative exists to this, if any...

- Agetian

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 29 Sep 2016, 19:21
by friarsol
Agetian wrote:
lujo wrote:Merseine always asks for 1 colorless instead of the enchanted creature's converted mana cost.
I can confirm this but I'm not sure how to fix it. The card seems to use a really unique expression <1\EnchantedCost> to refer to the enchanted creature's mana cost, but that doesn't seem to work (was it not coded?) and I don't know what kind of alternative exists to this, if any...

- Agetian
No there's definitely code in CostPartMana for it. And it was coded almost 2 years ago. Something recent must have broken it.

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 29 Sep 2016, 21:44
by samotnjak23
Any word on forge multiplayer? Server name?

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 30 Sep 2016, 00:25
by friarsol
samotnjak23 wrote:Any word on forge multiplayer? Server name?
For the most part it works, but your mileage may vary. The developer that was working on it hasn't been around in a while. You need one player to host, and another player to connect to the host.

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 30 Sep 2016, 05:41
by Agetian
friarsol wrote:
Agetian wrote:
lujo wrote:Merseine always asks for 1 colorless instead of the enchanted creature's converted mana cost.
I can confirm this but I'm not sure how to fix it. The card seems to use a really unique expression <1\EnchantedCost> to refer to the enchanted creature's mana cost, but that doesn't seem to work (was it not coded?) and I don't know what kind of alternative exists to this, if any...

- Agetian
No there's definitely code in CostPartMana for it. And it was coded almost 2 years ago. Something recent must have broken it.
Hmm, not sure yet what to think here - I tried debugging it and it look like the "restriction" parameter is passed as "null" when CostPartMana is created for Merseine :/

Also, isEnchantedCreatureCost() returns "false" at CostPartMana.java:125 for some reason, even though it *is* set up as "true" beforehand. Debugging this is not easy, tbh... Can someone take a look given enough time?..

- Agetian

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 30 Sep 2016, 11:36
by indicatie
Hmmm.... Never played it before, but Temporal Aperture doesn't seem to work. Or am I doing something wrong? I click the top card of my library and the Temporal Aperture effect "card" in my command zone, but nothing seems to happen.

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 30 Sep 2016, 14:05
by gos
. Madcap Experiment doesn't deal damage if it doesn't reveal an artifact.

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 30 Sep 2016, 14:26
by Hanmac
gos wrote:. Madcap Experiment doesn't deal damage if it doesn't reveal an artifact.
fixed with r32240.
did use wrong Remember Parameter, RememberRevealed is the right one.

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 30 Sep 2016, 22:05
by gos
. Garrulous Sycophant is draining 1 life every turn when no one is the monarch.

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 01 Oct 2016, 09:55
by tasteful
power and toughness changes from enchantment auras display on bestow creatures like Hopeful Eidolon, but don't actually work.

Re: Forge Beta Release: 09-26-2016 ver 1.5.56 (Kaladesh)

PostPosted: 02 Oct 2016, 01:15
by Xitax
Bug - Built to Smash gave trample to my non-artifact creature Voltaic Brawler