Collectible Card Game Headquarters

friarsol wrote:

Sloth wrote:I will try to implement a "MayLookAt" static ability to get Bane Alley Broker scripted.

Would love to see Sphinx of Jwar Isle converted if you get this figured out.

Unfortunately, Sphinx of Jwar Isle is more of an UI issue. For now I simply dealt with displaying face down cards in exile.

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.

Ahh that's too bad. How about Gustha's Scepter?

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.

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

Side effect: Hotseat matches are now possible.

This mode has some known bugs to fix like "play land from opponent's hand"

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

I'm going to convert all the slowtrips to delayed triggers over the next few days.

RedDeckWins wrote:I'm going to convert all the slowtrips to delayed triggers over the next few days.

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.

Hey RDW, did you skip Arcane Denial on purpose or just miss it?

Using r21788 the game freezes when Howling Mine triggers during the opponents turn. I guess a call to chooseOrderOfSimultaneousStackEntryAll is missing somewhere.

friarsol wrote:Hey RDW, did you skip Arcane Denial on purpose or just miss it?

I did all the brain dead ones first. Gonna convert the non-slowtrip but using draw | nextupkeep today.

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 wrote:

friarsol wrote:Hey RDW, did you skip Arcane Denial on purpose or just miss it?

I did all the brain dead ones first. Gonna convert the non-slowtrip but using draw | nextupkeep today.

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.

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.

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

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.

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.

RedDeckWins wrote:

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.

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.

Can we remember player on that delayed trigger?

Why can't you just add a parameter to existing 'draw' effect?