Page 2 of 15

Re: Forge Release 07/14/2020 ver 1.6.36

PostPosted: 21 Jul 2020, 11:34
by Diena
you are right, my mistake.

Re: Forge Release 07/14/2020 ver 1.6.36

PostPosted: 22 Jul 2020, 05:48
by LisaAlsop
Fregnor wrote:Read the Tides only gives the first of two options:
*Draw three cards.
*Return up to two target creatures to their owners' hands.

So it is only possible to draw cards and not possible to return creatures using this card.
thanks for your replies

Re: Forge Release 07/14/2020 ver 1.6.36

PostPosted: 23 Jul 2020, 03:43
by Fregnor
The Alternate Win Condition Achievement "The Flurry of Assassins" relating to Vraska, Golgari Queen cannot be earned when you meet the conditions to earn it.

In the past when reporting this issue, people have speculated that the triggers aren't tripping because you don't actually win the game, your opponent just loses. And since you have to win the game to earn an achievement, the coding is not set up right to give a trophy when you didn't win, but were still standing after your opponent lost.

A quick and dirty way to fix this problem would be to remove the achievement so it is not offered and people won't see it and won't try to earn it.

Re: Forge Release 07/14/2020 ver 1.6.36

PostPosted: 23 Jul 2020, 07:11
by indicatie
The AI has a Wandering Fumarole in play and activates it loads of times before giving up and not blocking with it, while I attack with a 2/1 Ape Pet.

Re: Forge Release 07/14/2020 ver 1.6.36

PostPosted: 23 Jul 2020, 13:13
by Diena
LisaAlsop wrote:
Fregnor wrote:Read the Tides only gives the first of two options:
*Draw three cards.
*Return up to two target creatures to their owners' hands.

So it is only possible to draw cards and not possible to return creatures using this card.
thanks for your replies
the second option is present, it is just there is no text. If you select the line below the "Draw three cards" the second mode is chosen

Re: Forge Release 07/14/2020 ver 1.6.36

PostPosted: 24 Jul 2020, 06:36
by N-Cat
Fregnor wrote:The Alternate Win Condition Achievement "The Flurry of Assassins" relating to Vraska, Golgari Queen cannot be earned when you meet the conditions to earn it.

In the past when reporting this issue, people have speculated that the triggers aren't tripping because you don't actually win the game, your opponent just loses. And since you have to win the game to earn an achievement, the coding is not set up right to give a trophy when you didn't win, but were still standing after your opponent lost.

A quick and dirty way to fix this problem would be to remove the achievement so it is not offered and people won't see it and won't try to earn it.
I reported this issue at the end of the last thread; an issue was opened up on GitLab, and they've already put in a merge request addressing the issue.

Additionally, that doesn't seem to have been the reason. Some of the other Alt Win achievements use "Lose the Game" effects, rather than "Win the game" effects (e.g. "Chance for Glory", "Door to Nothingness"). The currently believed reason—which the fix is addressing—is that the Lose trigger comes from the Emblem, but the achievement was checking for an effect coming from the card itself.

Re: Forge Release 07/14/2020 ver 1.6.36

PostPosted: 25 Jul 2020, 03:14
by Fregnor
Diena wrote:
LisaAlsop wrote:
Fregnor wrote:Read the Tides only gives the first of two options:
*Draw three cards.
*Return up to two target creatures to their owners' hands.

So it is only possible to draw cards and not possible to return creatures using this card.
thanks for your replies
the second option is present, it is just there is no text. If you select the line below the "Draw three cards" the second mode is chosen
Thanks for the clue! I figured out that the code needs to be like this:
Code: Select all
Name:Read the Tides
ManaCost:5 U
Types:Sorcery
A:SP$ Charm | Cost$ 5 U | Choices$ DBDraw,DBReturn
SVar:DBDraw:DB$ Draw | NumCards$ 3 | SpellDescription$ Draw three cards.
SVar:DBReturn:DB$ ChangeZone | TargetMin$ 0 | TargetMax$ 2 | ValidTgts$ Creature | TgtPrompt$ Select target creature | Origin$ Battlefield | Destination$ Hand | SpellDescription$ Return up to two target creatures to their owners' hands.
Oracle:Choose one —\n• Draw three cards.\n• Return up to two target creatures to their owners' hands.
It works correctly for me now in playtesting. The SVar:DBReturn:DB$ sentence was missing the SpellDescription$ command, and there was a typo where it said "Draw three card." rather than "Draw three cards."

Re: Forge Release 07/14/2020 ver 1.6.36

PostPosted: 25 Jul 2020, 08:56
by martinisam
It appears that See the Truth is still placing the unselected cards on top of the library instead of the bottom...

Re: Forge Release 07/14/2020 ver 1.6.36

PostPosted: 27 Jul 2020, 18:55
by MrMystery314
The game freezes when the AI plays Contagion.

Re: Forge Release 07/14/2020 ver 1.6.36

PostPosted: 28 Jul 2020, 04:46
by Deyool
Eureka is bugged for a long time now...
It is putting all cards at same time, and not one after another, that causa that effects like Doubling Season don't apply double counters or tokens to any card put after Doubling Season by the effect of Eureka.

Re: Forge Release 07/14/2020 ver 1.6.36

PostPosted: 28 Jul 2020, 13:25
by gos
I got this error when Mana Web resolved.

Code: Select all
java.lang.NullPointerException
   at forge.game.card.Card.canProduceSameManaTypeWith(Card.java:2358)
   at forge.game.card.CardProperty.cardHasProperty(CardProperty.java:879)
   at forge.game.card.Card.hasProperty(Card.java:4457)
   at forge.game.card.Card.isValid(Card.java:4446)
   at forge.game.GameObject.isValid(GameObject.java:27)
   at forge.game.card.CardPredicates$18.apply(CardPredicates.java:205)
   at forge.game.card.CardPredicates$18.apply(CardPredicates.java:202)
   at com.google.common.collect.Iterators$5.computeNext(Iterators.java:637)
   at com.google.common.collect.AbstractIterator.tryToComputeNext(AbstractIterator.java:141)
   at com.google.common.collect.AbstractIterator.hasNext(AbstractIterator.java:136)
   at forge.util.collect.FCollection.addAll(FCollection.java:291)
   at forge.util.collect.FCollection.<init>(FCollection.java:80)
   at forge.game.card.CardCollection.<init>(CardCollection.java:115)
   at forge.game.card.CardLists.filter(CardLists.java:297)
   at forge.game.card.CardLists.getValidCards(CardLists.java:208)
   at forge.game.ability.AbilityUtils.filterListByType(AbilityUtils.java:922)
   at forge.game.ability.effects.TapAllEffect.resolve(TapAllEffect.java:50)
   at forge.game.spellability.AbilitySub.resolve(AbilitySub.java:111)
   at forge.game.ability.AbilityUtils.resolveApiAbility(AbilityUtils.java:1415)
   at forge.game.ability.AbilityUtils.resolve(AbilityUtils.java:1383)
   at forge.ai.ComputerUtil.playNoStack(ComputerUtil.java:294)
   at forge.ai.PlayerControllerAi.playSpellAbilityNoStack(PlayerControllerAi.java:488)
   at forge.game.trigger.WrappedAbility.resolve(WrappedAbility.java:473)
   at forge.game.ability.AbilityUtils.resolveApiAbility(AbilityUtils.java:1415)
   at forge.game.ability.AbilityUtils.resolve(AbilityUtils.java:1383)
   at forge.game.zone.MagicStack.resolveStack(MagicStack.java:479)
   at forge.game.phase.PhaseHandler.startFirstTurn(PhaseHandler.java:1075)
   at forge.game.GameAction.startGame(GameAction.java:1626)
   at forge.game.Match.startGame(Match.java:90)
   at forge.match.HostedMatch$2.run(HostedMatch.java:245)
   at java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
   at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
   at java.lang.Thread.run(Unknown Source)

Re: Forge Release 07/14/2020 ver 1.6.36

PostPosted: 29 Jul 2020, 03:00
by Northmoc1344
Thanks for all the posts... very helpful!

Vraska, Golgari Queen issue has been re-opened.

Wandering Fumarole issue is here... hopefully someone tackles it soon :)

Read the Tides issues have all been addressed on the master version and should be fixed in all upcoming snapshots and releases.

See the Truth fix is in the pipe.

Will look into the newer reports soon. Thanks again.

Re: Forge Release 07/14/2020 ver 1.6.36

PostPosted: 30 Jul 2020, 02:50
by Fregnor
Dire Fleet Warmonger should be tagged as a card that the AI cannot play well using the line of code:
Code: Select all
AI:RemoveDeck:All
The problem is that the AI sacrifices creatures every turn to get the +2/+2 buff and Trample, but then often will not attack and will waste the sacrifice. It will even sacrifice itself when it is the last thing available to sacrifice.

So this card needs to be tagged so the AI will not chose it in a draft and so players will get warnings when giving a constructed deck containing it to the AI to play.

Re: Forge Release 07/14/2020 ver 1.6.36

PostPosted: 01 Aug 2020, 19:03
by peepser
AI casts Angelic Ascension on my Sabertooth Mauler (3/3), turning it into a 4/4 flyer. This doesn't make any sense.

Re: Forge Release 07/14/2020 ver 1.6.36

PostPosted: 02 Aug 2020, 19:48
by rex4
AI keeps taping for no reason Jegantha, the Wellspring at least in momir, I guess it doesn't know what "nongeneric mana" mean and tries to tap it to get an X= 5 more. If you get it it also allows you to cheat if say you have 6 lands + jegantha and tries to x=11 and press auto twice (you even still keep the 5 non generic floating manas) but nothing forces you to do it so that's okay.

By the way, I found bugs in many older cards (I only reported a few of them, the rest is on my computer) but it seems that you don't fix them so in fact I'm reporting this one quickly before it gets too old for you lol