myk's code contributions
Post MTG Forge Related Programming Questions Here
	Moderators: timmermac, Agetian, friarsol, Blacksmith, KrazyTheFox, CCGHQ Admins
Re: myk's code contributions
by friarsol » 17 Mar 2013, 21:23 
Didn't realize you couldn't update things yourself. You've been granted developer role so you can close issues for yourself now.myk wrote:I think we can close issue 634 on Mantis.
- friarsol
 - Global Moderator
 - Posts: 7593
 - Joined: 15 May 2010, 04:20
 - Has thanked: 243 times
 - Been thanked: 965 times
 
Re: myk's code contributions
by goonjamin » 17 Mar 2013, 21:26 
Hi myk, are skins and sounds going to be moved over to the non program location at some point as well?
			
		Re: myk's code contributions
by Max mtg » 17 Mar 2013, 21:28 
I expect it to work again with r20432 applied.myk wrote:Edit: it looks like the germ token added by Batterskull is still not appearing correctly.
Single class for single responsibility.
		- Max mtg
 - Programmer
 - Posts: 1997
 - Joined: 02 Jul 2011, 14:26
 - Has thanked: 173 times
 - Been thanked: 334 times
 
Re: myk's code contributions
by myk » 17 Mar 2013, 21:31 
@goonjamin: Some discussion and planning is in order first, but yes, I believe it will.
@Max: Thanks!
			
		@Max: Thanks!
- myk
 - Posts: 439
 - Joined: 17 Jan 2013, 02:39
 - Location: California
 - Has thanked: 38 times
 - Been thanked: 57 times
 
Re: myk's code contributions
by Max mtg » 17 Mar 2013, 21:33 
@Diogenes I also keep compete sets downloaded, but I made Forge use pictures right from that folders.
This is the setup I am using:
			This is the setup I am using:
- Code: Select all
 cardPicsDir=D:\\Projects\\mtg\\img-full
cardPicsSubDirs=LEA->LEA|LEB->LEB|2ED->2ED|ARN->ARN|ATQ->ATQ|3ED->3ED|LEG->LEG|DRK->DRK|FEM->FEM|4ED->4ED|ICE->ICE|CHR->CHR|HML->HML|ALL->ALL|MIR->MIR|VIS->VIS|5ED->5ED|POR->POR|WTH->WTH|TMP->TMP|STH->STH|EXO->EXO|PO2->P02|USG->USG|ULG->ULG|6ED->6ED|UDS->UDS|PTK->PTK|S99->S99|MMQ->MMQ|NMS->NEM|S00->S00|PCY->PCY|INV->INV|PLS->PLS|7ED->7ED|APC->APC|ODY->ODY|TOR->TOR|JUD->JUD|ONS->ONS|LGN->LGN|SCG->SCG|8ED->8ED|MRD->MRD|DST->DST|5DN->5DN|9ED->9ED|GPT->GPT|CSP->CSP|HOP->PCH|MBP->PR
Single class for single responsibility.
		- Max mtg
 - Programmer
 - Posts: 1997
 - Joined: 02 Jul 2011, 14:26
 - Has thanked: 173 times
 - Been thanked: 334 times
 
Re: myk's code contributions
by myk » 17 Mar 2013, 21:42 
+1 This is certainly easier than maintaining a supplemental card list : )Max mtg wrote:@Diogenes I also keep compete sets downloaded, but I made Forge use pictures right from that folders.
This is the setup I am using:
- Code: Select all
 cardPicsDir=D:\\Projects\\mtg\\img-full
cardPicsSubDirs=LEA->LEA|LEB->LEB|2ED->2ED|ARN->ARN|ATQ->ATQ|3ED->3ED|LEG->LEG|DRK->DRK|FEM->FEM|4ED->4ED|ICE->ICE|CHR->CHR|HML->HML|ALL->ALL|MIR->MIR|VIS->VIS|5ED->5ED|POR->POR|WTH->WTH|TMP->TMP|STH->STH|EXO->EXO|PO2->P02|USG->USG|ULG->ULG|6ED->6ED|UDS->UDS|PTK->PTK|S99->S99|MMQ->MMQ|NMS->NEM|S00->S00|PCY->PCY|INV->INV|PLS->PLS|7ED->7ED|APC->APC|ODY->ODY|TOR->TOR|JUD->JUD|ONS->ONS|LGN->LGN|SCG->SCG|8ED->8ED|MRD->MRD|DST->DST|5DN->5DN|9ED->9ED|GPT->GPT|CSP->CSP|HOP->PCH|MBP->PR
- myk
 - Posts: 439
 - Joined: 17 Jan 2013, 02:39
 - Location: California
 - Has thanked: 38 times
 - Been thanked: 57 times
 
Re: myk's code contributions
by Max mtg » 17 Mar 2013, 21:44 
These don't look like user-created data to be kept in personal profile.goonjamin wrote:Hi myk, are skins and sounds going to be moved over to the non program location at some point as well?
Why do you want them moved?
Single class for single responsibility.
		- Max mtg
 - Programmer
 - Posts: 1997
 - Joined: 02 Jul 2011, 14:26
 - Has thanked: 173 times
 - Been thanked: 334 times
 
Re: myk's code contributions
by myk » 17 Mar 2013, 21:45 
Once they are moved out of our distributable and onto the server, they would go into cacheDir, along with the other downloaded data.
			
		- myk
 - Posts: 439
 - Joined: 17 Jan 2013, 02:39
 - Location: California
 - Has thanked: 38 times
 - Been thanked: 57 times
 
Re: myk's code contributions
by myk » 17 Mar 2013, 21:57 
There are some pics in here with a '2' suffix. Could you possibly choose one pic per set?Xitax wrote:Attached is my current set of booster pics.
- myk
 - Posts: 439
 - Joined: 17 Jan 2013, 02:39
 - Location: California
 - Has thanked: 38 times
 - Been thanked: 57 times
 
Re: myk's code contributions
by Max mtg » 17 Mar 2013, 22:05 
Oh, by the way, the modern sets feature up to 5 booster pictures.
So, out Boosters might also obtain the artIndex field
			So, out Boosters might also obtain the artIndex field
Single class for single responsibility.
		- Max mtg
 - Programmer
 - Posts: 1997
 - Joined: 02 Jul 2011, 14:26
 - Has thanked: 173 times
 - Been thanked: 334 times
 
Re: myk's code contributions
by goonjamin » 17 Mar 2013, 22:07 
r20436 - The Download Card Prices isn't working on my system.  It just sits and sits when I click start but doesn't download anything.  Is this working for everyone else?
			
		Re: myk's code contributions
by myk » 17 Mar 2013, 22:15 
@Max: good point.  @Xitax, do you think you could get a hold of the complete set of pics?
@goonjamin, could you try reverting to r20422 and seeing if that fixes the problem? I just checked.  This was it.  Max: could you check your usage of java.nio classes in the downloader? My bad. will fix.
			
				Last edited by myk on 17 Mar 2013, 22:28, edited 1 time in total.
					
				
			
		- myk
 - Posts: 439
 - Joined: 17 Jan 2013, 02:39
 - Location: California
 - Has thanked: 38 times
 - Been thanked: 57 times
 
Re: myk's code contributions
by Max mtg » 17 Mar 2013, 22:22 
It's your r20428 that broke downloader window update when download is over.myk wrote:@goonjamin, could you try reverting to r20422 and seeing if that fixes the problem?I just checked. This was it. Max: could you check your usage of java.nio classes in the downloader?
line 350
Single class for single responsibility.
		- Max mtg
 - Programmer
 - Posts: 1997
 - Joined: 02 Jul 2011, 14:26
 - Has thanked: 173 times
 - Been thanked: 334 times
 
- myk
 - Posts: 439
 - Joined: 17 Jan 2013, 02:39
 - Location: California
 - Has thanked: 38 times
 - Been thanked: 57 times
 
Who is online
Users browsing this forum: No registered users and 101 guests