Page 2 of 3

Re: MTGForge 10/07 (unofficial BETA) version

PostPosted: 08 Oct 2009, 17:15
by DennisBergkamp
Thanks Huggy, I've added it to the first post :)

Re: MTGForge 10/07 (unofficial BETA) version

PostPosted: 08 Oct 2009, 19:05
by mtgrares
Thanks Dennis and to everyone that helped. I have to admit seeing MTG Forge with more than 2,000 cards is really mind-blowing (in a good way). Modular sounds fun.

Here is a zipped version that includes Chris's updates for quest and rarity.

10-07-forge.zip

Re: MTGForge 10/07 (unofficial BETA) version

PostPosted: 08 Oct 2009, 19:54
by mtgrares
I love that the computer can play counterspell and similar spells. Here is a list of cards the AI doesn't play, I presume SpellAbility.canPlayAI() returns false. All of the other spells like counterspell worked fine.

Dismiss, Cancel, Essence Scatter, Exclude, Overwhelming Intellect

Re: MTGForge 10/07 (unofficial BETA) version

PostPosted: 08 Oct 2009, 20:22
by DennisBergkamp
Hmm, strange... Exclude definitely works for me (check screenshot). What test showed you that the AI doesn't play those?

EDIT: Essence Scatter definitely doesn't work (for some reason I added "Essence Shatter" and "Essence Strike" - probably from Preemptive Strike - as names of counterspells for the AI to use). By the way, all of the counterspells will return false in canPlayAI(), but there's a forced check in MagicStack.java whenever a spell gets played, the AI will attempt to counter it.

Re: MTGForge 10/07 (unofficial BETA) version

PostPosted: 08 Oct 2009, 20:25
by mtgrares
Oops, sorry. I made an AI test deck with each one of the cards, 4 Islands 4 Exclude, and played creatures.

Re: MTGForge 10/07 (unofficial BETA) version

PostPosted: 08 Oct 2009, 20:31
by DennisBergkamp
Strange, then it should work. As long as the creature's converted manacost is equal or larger than the converted manacost of the counterspell (3 in the case of exclude).
Overwhelming Intellect should also work on 3.

Re: MTGForge 10/07 (unofficial BETA) version

PostPosted: 08 Oct 2009, 23:34
by Chris H.
The Hunted Horror creates two 3/3 green Centaur creature tokens with protection from black. We already have a card which creates a green 3/3 centaur that does not have protection from black. How do we want to handle this?

The Hunted Phantasm creates five 1/1 red Goblin creature tokens. We also have another card which creates red 1/1 goblin tokens. Neither have any special abilities and I guess that for the moment we can use the same jpg pic for both.

Re: MTGForge 10/07 (unofficial BETA) version

PostPosted: 09 Oct 2009, 01:09
by Chris H.
OK, we have three cards which creates a token:

Cloudseeder, Llanowar Mentor and Goldmeadow Lookout.

I checked online and the token jpgs are not available. The high quality scans located elsewhere on the CCGH web site should include these tokens for those who want to fill these holes.

The jpg names are:

u_1_1_cloud_sprite.jpg
g_1_1_llanowar_elves.jpg
w_1_1_goldmeadow_harrier.jpg

Re: MTGForge 10/07 (unofficial BETA) version

PostPosted: 09 Oct 2009, 06:03
by Marek14
Chris H. wrote:OK, we have three cards which creates a token:

Cloudseeder, Llanowar Mentor and Goldmeadow Lookout.

I checked online and the token jpgs are not available. The high quality scans located elsewhere on the CCGH web site should include these tokens for those who want to fill these holes.

The jpg names are:

u_1_1_cloud_sprite.jpg
g_1_1_llanowar_elves.jpg
w_1_1_goldmeadow_harrier.jpg
Don't we also have Sparkspitter and Sliversmith? If we have those, why not the last card of the cycle, Skirk Ridge Exhumer?

So, I guess there's also r_3_1_spark_elemental.jpg and c_1_1_metallic_sliver.jpg?

Re: MTGForge 10/07 (unofficial BETA) version

PostPosted: 09 Oct 2009, 10:59
by Chris H.
Marek14 wrote:Don't we also have Sparkspitter and Sliversmith? If we have those, why not the last card of the cycle, Skirk Ridge Exhumer?

So, I guess there's also r_3_1_spark_elemental.jpg and c_1_1_metallic_sliver.jpg?
`
Hmmm, good question.

In our discussions about the tokens created by the three cards Cloudseeder, Llanowar Mentor and Goldmeadow Lookout we may have missed the Sparkspitter and the Sliversmith.

Re: MTGForge 10/07 (unofficial BETA) version

PostPosted: 09 Oct 2009, 14:45
by DennisBergkamp
Yes, Sliversmith should produce tokens with an image name "C 1 1 Metallic Sliver".
We don't have Sparkspitter, because for some reason the code for this card is completely different. Instead of creating a token, it makes a direct copy of the existing Spark Elemental card. Anyway, I'll change the code for this, and set its name to "R 3 1 Spark Elemental".

Re: MTGForge 10/07 (unofficial BETA) version

PostPosted: 09 Oct 2009, 17:11
by DennisBergkamp
The Hunted Horror creates two 3/3 green Centaur creature tokens with protection from black. We already have a card which creates a green 3/3 centaur that does not have protection from black. How do we want to handle this?
I'll give them an imageName of "G 3 3 Centaur Pro Black" => g_3_3_centaur_pro_black.jpg.

Re: MTGForge 10/07 (unofficial BETA) version

PostPosted: 09 Oct 2009, 18:00
by mtgrares
DennisBergkamp wrote:Strange, then it should work. As long as the creature's converted manacost is equal or larger than the converted manacost of the counterspell (3 in the case of exclude).
Overwhelming Intellect should also work on 3.
I know what the problem was, the manacost wasn't equal or larger than the converted manacost of the counterspell. I didn't look at the source code and I presumed that the computer just played counterspells willy-nilly. I like the heuristic that looks at the mana cost of the counterspell spell and the card being played.

Re: MTGForge 10/07 (unofficial BETA) version

PostPosted: 11 Oct 2009, 03:31
by kingbob
Some notes:

-I played Joiner Adept and it correctly changed my lands, but when it was destroyed my land were still able to produce all colors.

-Manlands won't produce mana; either will Ravnica dual lands

Re: MTGForge 10/07 (unofficial BETA) version

PostPosted: 12 Oct 2009, 00:59
by zerker2000
kingbob wrote:-I played Joiner Adept and it correctly changed my lands, but when it was destroyed my land were still able to produce all colors.
Ah, I recently predicted this bug in developer corner, working on it :P
-Manlands won't produce mana; either will Ravnica dual lands
Manlands? We have Ravnica lands? Examples please...