Bug Reports (snapshot builds)
Post MTG Forge Related Programming Questions Here
Moderators: timmermac, Blacksmith, KrazyTheFox, Agetian, friarsol, CCGHQ Admins
Re: Bug Reports (snapshot builds)
by stormcat » 01 Jun 2016, 04:42
In Eternal Masters, Xantid Swarm's Rarity is wrong.
Not Common, it's Rare.
Not Common, it's Rare.
Re: Bug Reports (snapshot builds)
by stormcat » 01 Jun 2016, 06:47
:r31258
I found another mistake in Eternal Masters.
Ticking Gnomes R→U
Yavimaya Enchantress R→C
Sensei's Divining Top U→R
Winter Orb U→R
Prismatic Lens C→U
I found another mistake in Eternal Masters.
Ticking Gnomes R→U
Yavimaya Enchantress R→C
Sensei's Divining Top U→R
Winter Orb U→R
Prismatic Lens C→U
Re: Bug Reports (snapshot builds)
by Hanmac » 01 Jun 2016, 08:47
@stormcat: thanks for pointing that out for me, i had some typos while i did type it from hand.
===
r31259:
Problem with SplitCards:
1) if i cast them with Fuse: it does only show the color of the left card but it should show the combined. so while on the stack i cant see from the color if the spell is red or not.
2) while in Deck Editor Splitcards are shown with the Transform Swirl, while in the game they do not. Imo they should either don't show the swirl for them or the first page should be the left one and not the combined. (and even if they show the comined one, they show to have the wrong color)
3) Interaction with Harness the Storm: i can only use it when i cast the spell with Fuse. that has something to do with sharesNameWith, it should look at both Sides. Also when i cast a Spell with Fuse with Harness the Storm, i can select to cast it with Fuse what should not work.
(i asked a Judge about if Harness the Storm does work with split cards)
4) Other cost done with Harness the Storm like Surge and Awaken should work, but currently they do not.
===
Cards with Alternate State like Split cards:
Naming a Card for something Council of the Absolute does not work correclty for cards with Alternate State.
Alive // Well does appear twice, but when i select it the chosen card is "Alive // Well", but the ruling says:
also should be able to cast the other half of the named card. (if opponent does control it)
i don't know how Alhammarret, High Arbiter works with Split cards. i think i also should only be able to name one half of a split card. And my Opponent should be able to cast the other half.
This might be a problem from chooseCardName and later the magic Db with getMagicDb().getCommonCards().getUniqueCards();
===
r31259:
Problem with SplitCards:
1) if i cast them with Fuse: it does only show the color of the left card but it should show the combined. so while on the stack i cant see from the color if the spell is red or not.
2) while in Deck Editor Splitcards are shown with the Transform Swirl, while in the game they do not. Imo they should either don't show the swirl for them or the first page should be the left one and not the combined. (and even if they show the comined one, they show to have the wrong color)
3) Interaction with Harness the Storm: i can only use it when i cast the spell with Fuse. that has something to do with sharesNameWith, it should look at both Sides. Also when i cast a Spell with Fuse with Harness the Storm, i can select to cast it with Fuse what should not work.
(i asked a Judge about if Harness the Storm does work with split cards)
i have a fix for sharesNameWith but not commited yet.Magic Judge:You’ll be able to cast either half of Alive // Well from your graveyard with Harness the Storm’s trigger no matter which side(s) of it you cast from your hand.magichanmac asked: Question about interaction of Harness the Storm with Split Cards. For example with Alive // Well one in my hand, the other in Graveyard. Whatever i cast from my hand does it have consequences what i can cast from my graveyard? or can only cast something when i cast the spell with Fuse? (because otherwise it doesn't share a name with the one in the stack?) Also i know what i cant cast Fuse spells from my Graveyard with Fuse.
For example, you can cast Well, then use Harness’s trigger to cast Alive from your graveyard.
4) Other cost done with Harness the Storm like Surge and Awaken should work, but currently they do not.
===
Cards with Alternate State like Split cards:
Naming a Card for something Council of the Absolute does not work correclty for cards with Alternate State.
Alive // Well does appear twice, but when i select it the chosen card is "Alive // Well", but the ruling says:
means i should only be able to choice Alive or Well, not the combined one.4/15/2013: You can name either half of a split card, but not both. If you name half of a split card, your opponents can cast the other half. If that split card has fuse, your opponents can't cast that card as a fused split card.
also should be able to cast the other half of the named card. (if opponent does control it)
i don't know how Alhammarret, High Arbiter works with Split cards. i think i also should only be able to name one half of a split card. And my Opponent should be able to cast the other half.
This might be a problem from chooseCardName and later the magic Db with getMagicDb().getCommonCards().getUniqueCards();
Last edited by Hanmac on 01 Jun 2016, 14:14, edited 2 times in total.
Re: Bug Reports (snapshot builds)
by Marek14 » 01 Jun 2016, 10:11
Another bug with Psychogenic Probe: it's not triggered by Mindblaze, for some reason.
Stealing opponent's Dragonlord Silumgar should cause whatever permanent IT stole to be returned.
Stealing opponent's Dragonlord Silumgar should cause whatever permanent IT stole to be returned.
Re: Bug Reports (snapshot builds)
by Hanmac » 01 Jun 2016, 13:16
yeah i was surprised too and i added it.friarsol wrote:As far as I can tell, Mindblaze never actually shuffles a library (even though it should).
Sad that my problems with SplitCards are not that easy to fix.
Re: Bug Reports (snapshot builds)
by Hanmac » 01 Jun 2016, 16:31
Found this combo: Neck Breaker + Stonebrow, Krosan Hero
===
Edit:
Edit2:
seems Forge have some problems with "just in time" things ...
Forge currently doesn't support that. (it doesn't trigger for Neck Breaker)Magic Judge:Neck Breaker’s ability isn’t a triggered ability. It’s a static ability that applies to all attacking creatures. A creature without trample that attacks will gain trample from Neck Breaker in just enough time for Stonebrow to be able to trigger and give it an extra +2/+2.goat-shield asked: In my Stonebrow, Krosan Hero EDH deck, I'm wondering what the interaction with Neck Breaker would be... Are both abilities triggers that I can stack to my benefit?
(I had to go to @tabakrules for clarification on this one, as the rules as written aren’t clear here.)
===
Edit:
i implemented "sharesCMCWith" for Seasons Past, you guys might look at it or if it breaks something.Magic Judge:A split card not on the stack has two sets of characteristics, so it has two different mana costs and the associated converted mana costs.snowskeleton asked: Back on the split card thing, what is their CMC when not on the stack? For example, how do they interact with Seasons Past?
For Seasons Past, if you choose to return a split card, it takes the slots for both of its CMCs. So if you returned Armed // Dangerous, then you couldn’t return any other cards with CMCs of 2 or 4 to your hand with that Seasons Past.
Edit2:
Forge currently doesn't support that. similar to the first thing might have something to do with the trigger/checks :/Magic Judge:Yes.docmannyonly asked: If my Fleshbag Marauder is exiled by SilkWrap, and I have no other creatures on play, If I cast Compelling Deterrence Targeting silk wrap, would my opponent discard?
You get back Fleshbag Marauder immediately when Silkwrap leaves the battlefield. Fleshbag’s sacrifice trigger triggers when it returns, but that trigger doesn’t even go on the stack until after Compelling Deterrence finishes resolving. You control a zombie when that check is made and your opponent will discard a card.
seems Forge have some problems with "just in time" things ...
Last edited by Hanmac on 02 Jun 2016, 04:37, edited 1 time in total.
Re: Bug Reports (snapshot builds)
by Marek14 » 01 Jun 2016, 23:28
Freyalise's Winds / Seedborn Muse interaction is broken. Seems that Freyalise's Winds replaces any untap during untap step with removal of wind counters, but it should happen ONLY in its controller's untap step -- if the permanent untaps in another player's untap step, it should untap as normal.
Re: Bug Reports (snapshot builds)
by Hanmac » 02 Jun 2016, 06:40
fixed in r31263 with adding the Controller to the is Phase check.Marek14 wrote:Freyalise's Winds / Seedborn Muse interaction is broken. Seems that Freyalise's Winds replaces any untap during untap step with removal of wind counters, but it should happen ONLY in its controller's untap step -- if the permanent untaps in another player's untap step, it should untap as normal.
Re: Bug Reports (snapshot builds)
by Marek14 » 02 Jun 2016, 07:51
A bug that has persisted for a long time: when AI uses the first ability of Jace, the Mind Sculptor, I have no way to see whether it put the card on top or bottom of library.
Re: Bug Reports (snapshot builds)
by Hanmac » 02 Jun 2016, 10:40
Copy something with Madness cost:
r31264: casting Avacyn's Judgment for its madness cost, copy it with Pyromancer's Goggles should make a copy which does have copied Madness and X too. but currently it does not.
same true for Djinn Illuminatus and its Replicate.
i try to add CardFactory#copyCard this:
After some Investigate, the problem is that getXManaCostPaid of that Madness X spells. when i copy the spell with Pyromancer's Goggles it works, but with Replicate it did not.
Debug output did show me that getXManaCostPaid is not set yet when the spell is copied by/with Replicate, but is later when copied with Googles.
So it does look like that Replicate somehow does get done before the Mana is set correct ... i need to investigate more.
===
Also i noticed when i does press [cancel] for the Relicate it does UNTAP the Googles ... happens sometimes for other spells too.
r31264: casting Avacyn's Judgment for its madness cost, copy it with Pyromancer's Goggles should make a copy which does have copied Madness and X too. but currently it does not.
same true for Djinn Illuminatus and its Replicate.
i try to add CardFactory#copyCard this:
- Code: Select all
out.setMadness(in.isMadness());
After some Investigate, the problem is that getXManaCostPaid of that Madness X spells. when i copy the spell with Pyromancer's Goggles it works, but with Replicate it did not.
Debug output did show me that getXManaCostPaid is not set yet when the spell is copied by/with Replicate, but is later when copied with Googles.
So it does look like that Replicate somehow does get done before the Mana is set correct ... i need to investigate more.
===
Also i noticed when i does press [cancel] for the Relicate it does UNTAP the Googles ... happens sometimes for other spells too.
Last edited by Hanmac on 02 Jun 2016, 21:19, edited 1 time in total.
Re: Bug Reports (snapshot builds)
by Marek14 » 02 Jun 2016, 18:21
Spells that distribute damage (like Violent Eruption), don't seem to show the actual damage distribution anywhere when they are on stack, only the targets.
The ability of Mirror-Sigil Sergeant should be optional.
Strange bug happened when AI discarded Gibbering Descent to Kindle the Carnage; no damage was dealt and when AI didn't cast Gibbering Descent for madness cost, it stayed in exile permanently. Maybe it was caused by the fact that it was random discard?
The ability of Mirror-Sigil Sergeant should be optional.
Strange bug happened when AI discarded Gibbering Descent to Kindle the Carnage; no damage was dealt and when AI didn't cast Gibbering Descent for madness cost, it stayed in exile permanently. Maybe it was caused by the fact that it was random discard?
Re: Bug Reports (snapshot builds)
by Hanmac » 03 Jun 2016, 11:26
Marek14: fixed Mirror-Sigil Sergeant, the other things might be a bit more complicated. for the AI and madness thing, yeah it seems the AI does use the cards wrong and thats why the cards stay in exile.
===
hm i found "This Exalted handler should be converted to script" (some other things or TODOs might want to be scripted too) and i did try this in CardFactoryUtil:
i know this is not crittical, but for other or similar things which does add special Keywords to other cards we might have a common place so we don't need to implement everything twice.
hm i might implement it like Bushido, but thats not much different than it is currently.
===
hm i found "This Exalted handler should be converted to script" (some other things or TODOs might want to be scripted too) and i did try this in CardFactoryUtil:
- Code: Select all
// Exalted
final int exalted = card.getAmountOfKeyword("Exalted");
//card.removeIntrinsicKeyword("Exalted");
final StringBuilder trigExalted = new StringBuilder(
"Mode$ Attacks | ValidCard$ Creature.YouCtrl | Alone$ True | "
+ "Execute$ ExaltedPump | TriggerZones$ Battlefield | TriggerDescription$ "
+ "Exalted (Whenever a creature you control attacks alone, that creature "
+ "gets +1/+1 until end of turn.)");
final String abStringExalted = "DB$ Pump | Defined$ TriggeredAttacker | NumAtt$ +1 | NumDef$ +1";
card.getCurrentState().setSVar("ExaltedPump", abStringExalted);
final Trigger exaltedTrigger = TriggerHandler.parseTrigger(trigExalted.toString(), card, true);
for (int i = 0; i < exalted; i++) {
card.getCurrentState().addTrigger(exaltedTrigger);
} // Exalted
i know this is not crittical, but for other or similar things which does add special Keywords to other cards we might have a common place so we don't need to implement everything twice.
hm i might implement it like Bushido, but thats not much different than it is currently.
Re: Bug Reports (snapshot builds)
by Marek14 » 03 Jun 2016, 12:51
Suggestion: When a deck that includes Vanguard Avatar is selected in constructed (a few of gos's decks do), Forge should notify you and offer to switch to Vanguard mode.
Re: Bug Reports (snapshot builds)
by Hanmac » 04 Jun 2016, 08:16
something is funky with Autopayment with Mana Bloom.
if i pay with auto, it does remove more than one counter from it.
AI does cast Lavaball Trap for the Trap cost even i didn't play lands this turn.
if i pay with auto, it does remove more than one counter from it.
AI does cast Lavaball Trap for the Trap cost even i didn't play lands this turn.
Who is online
Users browsing this forum: No registered users and 38 guests