myk's code contributions
Post MTG Forge Related Programming Questions Here
Moderators: timmermac, Blacksmith, KrazyTheFox, Agetian, friarsol, 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 16 guests