Developer Plans
Post MTG Forge Related Programming Questions Here
Moderators: timmermac, Agetian, friarsol, Blacksmith, KrazyTheFox, CCGHQ Admins
Re: Developer Plans
by Sloth » 06 May 2013, 20:46
Unfortunately, Sphinx of Jwar Isle is more of an UI issue. For now I simply dealt with displaying face down cards in exile.friarsol wrote:Would love to see Sphinx of Jwar Isle converted if you get this figured out.Sloth wrote:I will try to implement a "MayLookAt" static ability to get Bane Alley Broker scripted.
-

Sloth - Programmer
- Posts: 3498
- Joined: 23 Jun 2009, 19:40
- Has thanked: 125 times
- Been thanked: 507 times
Re: Developer Plans
by friarsol » 06 May 2013, 23:11
Ahh that's too bad. How about Gustha's Scepter?Sloth wrote:Unfortunately, Sphinx of Jwar Isle is more of an UI issue. For now I simply dealt with displaying face down cards in exile.
- friarsol
- Global Moderator
- Posts: 7593
- Joined: 15 May 2010, 04:20
- Has thanked: 243 times
- Been thanked: 965 times
Re: Developer Plans
by Max mtg » 07 May 2013, 11:21
Sloth, looks like you've made it ... suboptimally.
1. There is already a function in Card - canBeShownTo. It appears you've added a duplicate 'canBeSeenBy'. Please condier merging them.
2. You can fetch current player controlled by GUI with a shorter method Singletons.getControl().getPlayer()
3. GuiChoose is a really bad place to reference CardFactory. If you want to display the card's other state the 2nd parameter of forge.gui.match.CMatchUI.setCard(Card, boolean) might be of help.
1. There is already a function in Card - canBeShownTo. It appears you've added a duplicate 'canBeSeenBy'. Please condier merging them.
2. You can fetch current player controlled by GUI with a shorter method Singletons.getControl().getPlayer()
3. GuiChoose is a really bad place to reference CardFactory. If you want to display the card's other state the 2nd parameter of forge.gui.match.CMatchUI.setCard(Card, boolean) might be of help.
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: Developer Plans
by Max mtg » 21 May 2013, 19:36
Plans - after being able to run ai vs ai I want to start a game with both players controlled by human.
After that change control over players dynamically - final aim is Mindslaver
After that change control over players dynamically - final aim is Mindslaver
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: Developer Plans
by Max mtg » 23 May 2013, 22:31
Side effect: Hotseat matches are now possible.
This mode has some known bugs to fix like "play land from opponent's hand"
This mode has some known bugs to fix like "play land from opponent's hand"
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: Developer Plans
by Agetian » 25 May 2013, 05:05
Sorry for disappearing again, fellas - had (and still have) a couple of projects on my hands that demand immediate attention, so, sadly, can't get immediately back to coding Forge, but will slowly start integrating it into my schedule, so expect some svn commits slowly starting from me soon (still heading for the same goal as previously delineated - working on AI personalities). Checked out the latest SVN build, made everything compile on my PC, looks good so far, so I should be ready to go. Congrats to everyone for some major achievements during my absence - the AI vs. AI / hotseat matches look interesting, nice job on DGM cards and their integration, overall I enjoy the latest Forge! 
- Agetian
- Agetian
- Agetian
- Programmer
- Posts: 3490
- Joined: 14 Mar 2011, 05:58
- Has thanked: 684 times
- Been thanked: 572 times
Re: Developer Plans
by RedDeckWins » 25 May 2013, 20:03
I'm going to convert all the slowtrips to delayed triggers over the next few days.
-

RedDeckWins - Posts: 35
- Joined: 20 Apr 2013, 16:45
- Has thanked: 8 times
- Been thanked: 10 times
Re: Developer Plans
by Max mtg » 25 May 2013, 20:24
Would be awesome if you do. I never liked that slowtrip member in Player class.RedDeckWins wrote:I'm going to convert all the slowtrips to delayed triggers over the next few days.
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: Developer Plans
by friarsol » 28 May 2013, 00:29
Hey RDW, did you skip Arcane Denial on purpose or just miss it?RedDeckWins wrote:I'm going to convert all the slowtrips to delayed triggers over the next few days.
- friarsol
- Global Moderator
- Posts: 7593
- Joined: 15 May 2010, 04:20
- Has thanked: 243 times
- Been thanked: 965 times
Re: Developer Plans
by Sloth » 28 May 2013, 18:56
Using r21788 the game freezes when Howling Mine triggers during the opponents turn. I guess a call to chooseOrderOfSimultaneousStackEntryAll is missing somewhere.
-

Sloth - Programmer
- Posts: 3498
- Joined: 23 Jun 2009, 19:40
- Has thanked: 125 times
- Been thanked: 507 times
Re: Developer Plans
by RedDeckWins » 29 May 2013, 01:57
I did all the brain dead ones first. Gonna convert the non-slowtrip but using draw | nextupkeep today.friarsol wrote:Hey RDW, did you skip Arcane Denial on purpose or just miss it?
This is what I have left:
arcane denial
foreshadow
library of lat nam
lodestone bauble
sapphire charm
thermal flux
This is the regex I used, nothing special: Draw.*NextUpkeep
If there's other cards outside that list, let me know.
-

RedDeckWins - Posts: 35
- Joined: 20 Apr 2013, 16:45
- Has thanked: 8 times
- Been thanked: 10 times
Re: Developer Plans
by swordshine » 29 May 2013, 05:41
Arcane Denial has an effect "draw up to two cards", it allows you to draw one card. The current parameter "OptionalDecider" in the Draw effect need to be updated.RedDeckWins wrote:I did all the brain dead ones first. Gonna convert the non-slowtrip but using draw | nextupkeep today.friarsol wrote:Hey RDW, did you skip Arcane Denial on purpose or just miss it?
This is what I have left:
arcane denial
foreshadow
library of lat nam
lodestone bauble
sapphire charm
thermal flux
This is the regex I used, nothing special: Draw.*NextUpkeep
If there's other cards outside that list, let me know.
- swordshine
- Posts: 682
- Joined: 11 Jul 2010, 02:37
- Has thanked: 116 times
- Been thanked: 87 times
Re: Developer Plans
by Max mtg » 29 May 2013, 13:04
I have renamed MatchState (aka MatchController) and GameState to plain 'Match' and "Game" to make things simplier
Please pay attention if any of your non-commited code fails to compile
Please pay attention if any of your non-commited code fails to compile
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: Developer Plans
by RedDeckWins » 29 May 2013, 13:16
I noticed that as well. Both arcane denial and diminishing returns need a new "draw up to" effect. However, for the slowtrip conversion, arcane denial is working how it always has but with delayed triggers.swordshine wrote:Arcane Denial has an effect "draw up to two cards", it allows you to draw one card. The current parameter "OptionalDecider" in the Draw effect need to be updated.
-

RedDeckWins - Posts: 35
- Joined: 20 Apr 2013, 16:45
- Has thanked: 8 times
- Been thanked: 10 times
Re: Developer Plans
by Max mtg » 29 May 2013, 13:41
Can we remember player on that delayed trigger?RedDeckWins wrote:I noticed that as well. Both arcane denial and diminishing returns need a new "draw up to" effect. However, for the slowtrip conversion, arcane denial is working how it always has but with delayed triggers.swordshine wrote:Arcane Denial has an effect "draw up to two cards", it allows you to draw one card. The current parameter "OptionalDecider" in the Draw effect need to be updated.
Why can't you just add a parameter to existing 'draw' effect?
Single class for single responsibility.
- Max mtg
- Programmer
- Posts: 1997
- Joined: 02 Jul 2011, 14:26
- Has thanked: 173 times
- Been thanked: 334 times
Who is online
Users browsing this forum: No registered users and 39 guests