Page 1 of 1

Commander set info

PostPosted: 04 Jul 2011, 13:19
by Chris H.
We have 19 cards that are new to the Commander set. Set info script is adding the correct data to these card files. For example, Animar, Soul of Elements has the following:

SetInfo:COM|Mythic|http://magiccards.info/scans/en/cmd/181.jpg
`
I added the following line to the setdata.txt file:

Code: Select all
Index:76|Code2:COM|Code3:COM|Name:Commander
`
and these 19 new Commander cards display the set data in the deck editor. I can also sort by the Commander set but I only get the 19 new Commander cards. We have a number of pre-existing cards in forge that are also in Commander, but set info script is not adding in the Commander set info for the pre-existing cards.

Re: Commander set info

PostPosted: 04 Jul 2011, 13:45
by friarsol
Yep, this is what the reprintSetInfo.py script is for. I'll run it and commit the existing cards.

Re: Commander set info

PostPosted: 04 Jul 2011, 14:18
by friarsol
Apparently this was never done for NPH either, so that will be combined in my checkin to come.

Note to all: This script isn't very forgiving as it completely wipes out the existing SetInfo and then the SetInfo script is reran. This wipes away the multiple image data that exists for some older cards and Basic Lands. At some point I'll try to grab the info about multiple images in the SetInfoScript so we don't have to worry about losing information in the purge.

Re: Commander set info

PostPosted: 04 Jul 2011, 14:36
by Chris H.
Thank you Sol.

I will add in the line to the setdata.txt file once my update completes. :mrgreen:

Re: Commander set info

PostPosted: 04 Jul 2011, 14:51
by friarsol
Also, I'm not sure if the Commander pics are up on the server yet. I'm not sure how Rob handles that.

Re: Commander set info

PostPosted: 04 Jul 2011, 14:53
by Chris H.
friarsol wrote:Also, I'm not sure if the Commander pics are up on the server yet. I'm not sure how Rob handles that.
`
There was a message elsewhere on the CCGH forums stating that the HQ Commander pics are being worked on at this time. We may not have to wait too long.

Re: Commander set info

PostPosted: 04 Jul 2011, 16:29
by Rob Cashwalker
The setcode "COM" may not work well for Windows users.. suggest "CMD". I'll keep checking the pictures section for commander images, when it's ready I'll upload it.

Re: Commander set info

PostPosted: 04 Jul 2011, 19:49
by timmermac
Gatherer has card pictures up for Commander.

Re: Commander set info

PostPosted: 06 Jul 2011, 21:10
by friarsol
Looks like HQ pics are out: viewtopic.php?p=64447#p64447

Re: Commander set info

PostPosted: 06 Jul 2011, 23:08
by juzamjedi
CHAOS WARP!!!
Definitely want to add this to cube :D

Re: Commander set info

PostPosted: 07 Jul 2011, 03:09
by slapshot5
juzamjedi wrote:CHAOS WARP!!!
Definitely want to add this to cube
Should be scriptable for someone who wants to take the time.

Re: Commander set info

PostPosted: 07 Jul 2011, 10:24
by Sloth
slapshot5 wrote:
juzamjedi wrote:CHAOS WARP!!!
Definitely want to add this to cube
Should be scriptable for someone who wants to take the time.
Nope. There is the problem with Auras again.

Re: Commander set info

PostPosted: 07 Jul 2011, 22:43
by slapshot5
Sloth wrote:Nope. There is the problem with Auras again.
Bah! So there is. I think there's some code for Lost Auramancers that could be integrated into AF_ChangeZone with small changes to support this.

-slapshot5

Re: Commander set info

PostPosted: 07 Jul 2011, 23:59
by friarsol
slapshot5 wrote:
Sloth wrote:Nope. There is the problem with Auras again.
Bah! So there is. I think there's some code for Lost Auramancers that could be integrated into AF_ChangeZone with small changes to support this.

-slapshot5
Ideally, Attach should be the location of this (and reuse what's existing). An Aura ETB that wasn't cast with a target would just call one of these functions as it ETB to choose. Although we kinda have to get everything converted over to Attach before that makes sense.