AbilityFactory TODO list

[slapshot5]

I have renamed everything to "Dig". Documentation still forthcoming after parameter tweaks are finalized (for now).

-slapshot5

[slapshot5]

Re: Dig AF vs ChangeZone AF

Is something like Thoughtpicker Witch better implemented with modifications to ChangeZone or with Dig? I can implement it with no code changes in AF_Dig. (I submitted to SVN if you want to view.)

-slapshot5

[friarsol]

Re: Dig AF vs ChangeZone AF

Is something like Thoughtpicker Witch better implemented with modifications to ChangeZone or with Dig? I can implement it with no code changes in AF_Dig. (I submitted to SVN if you want to view.)

-slapshot5

To me that makes more sense to be in Dig. Digging is a refined version of ChangeZone where it's taking multiple cards from somewhere (usually the Library) and doing something to a few of them and doing something to the rest.

[slapshot5]

Starting on AF_Copy. See original post in other thread, but post follow-up/discussion here.

Ok. I'm feeling just stupid enough to try AF_Copy today. It's dreary, the coffee is ready, and the female is gone. After all, if I just copy the Kiki code, it can't be any worse than we already have, right?

Can someone post a list of the most basic test cards? Once the basics work, it can be expanded. As I said, I'll use Kiki as a template.

I suppose the AI is as simple as "copy best available valid card"?

-slapshot5

[Chris H.]

Starting on AF_Copy. See original post in other thread, but post follow-up/discussion here.

Ok. I'm feeling just stupid enough to try AF_Copy today. It's dreary, the coffee is ready, and the female is gone. After all, if I just copy the Kiki code, it can't be any worse than we already have, right?

Can someone post a list of the most basic test cards? Once the basics work, it can be expanded. As I said, I'll use Kiki as a template.

I suppose the AI is as simple as "copy best available valid card"?

`
We wish you luck.

[friarsol]

Starting on AF_Copy. See original post in other thread, but post follow-up/discussion here.

Ok. I'm feeling just stupid enough to try AF_Copy today. It's dreary, the coffee is ready, and the female is gone. After all, if I just copy the Kiki code, it can't be any worse than we already have, right?

Can someone post a list of the most basic test cards? Once the basics work, it can be expanded. As I said, I'll use Kiki as a template.

I suppose the AI is as simple as "copy best available valid card"?

-slapshot5

Here's a list that might be helpful.
http://magiccards.info/query?q=+-o%3Asp ... er&s=cname

[slapshot5]

Ok, I have the basics of an AF_CopyPermanent checked in. As we add more cards, more parameters will probably be necessary, and tweaks will undoubtedly need to happen.

Targeting/Defined is for Card objects.

Keywords param adds keywords to the created token
AtEOT param (can be Sacrifice or Exile) adds a sacrifice or exile at end of turn for the created token (not just a keyword so a little AI can be extracted...)

Please try to add a few new cards and see what parameters/tweaks we are going to need.

-slapshot5

edit: SacAtEOT -> AtEOT

[Hellfish]

Well, if you can implement extra good AI using a SacAtEOT parameter rather than keyword, an ExileAtEOT parameter would probably also be good for Splinter Twin, Mimic Vat and Heat Shimmer

[SoulStorm]

Turned on the computer this afternoon, checked the change log, and saw AF_CopyPermanent.

Here's a copy of my post from the log:

Dennis, I did not expect to see this today, but you have no idea how happy I am to see it! Not only have you implemented my most anticipated AF, but you just happened to implement the #1 card on my AF Copy list, Mirrorworks! I had no idea it was my birthday today.

[slapshot5]

Well, if you can implement extra good AI using a SacAtEOT parameter rather than keyword, an ExileAtEOT parameter would probably also be good for Splinter Twin, Mimic Vat and Heat Shimmer

Yep. I saw Heat Shimmer and implemented it as AtEOT param with values of Sacrifice and Exile.

-slapshot5

[Hellfish]

Cool stuff! One step further towards Mimic Vat, Prototype Portal & Soul Foundry

[friarsol]

Well if anyone is motivated to try their hand at 'CopySpell' now that they mastered the power of 'CopyPermanent' it would be great to have WheneverKeyword completely removed from the codebase. I'm taking a look to see if I can convert Mishra today.

[slapshot5]

Well if anyone is motivated to try their hand at 'CopySpell' now that they mastered the power of 'CopyPermanent' it would be great to have WheneverKeyword completely removed from the codebase. I'm taking a look to see if I can convert Mishra today.

Yep. That's why mine was CopyPermanent and not just Copy... I think Chris is bored, what with all those crazy big checkins I saw yesterday. Maybe he'll take a crack.

[Chris H.]

Yep. That's why mine was CopyPermanent and not just Copy... I think Chris is bored, what with all those crazy big checkins I saw yesterday. Maybe he'll take a crack.

`
Hehe.

Sloth and Corwin have done a great job of revamping existing and adding new quest decks. Need to give Sloth a break for real life stuff, and he will get back to Corwin and I once he can look at our most recent work.

Currently I am trying to finish adding spaces and changing the card real name with "CARDNAME" in SpellDescription$. I ran out of steam a few weeks ago when I started and I am now trying to finish this monster off.

I also want to change over the various ".getController().equals(AllZone.ComputerPlayer)" and ".getController().equals(AllZone.HumanPlayer)" to "getController().isComputer()" and "getController().isHuman()".

This is the kind of work that is just right for us jr. programmers in training.

I have also spent some time studying PhaseUtil.canOnlyUntapOneLand(). I thought that I might be able to create a CanOnlyUntap keyword for this ... but it looks like this is going to take me more time to figure out.

[slapshot5]

Ok. I'll look at AF_CopySpell today. I'm going to use the Kiki argument: "If I just copy Fork, it can't be worse than we already have!"

And maybe a little static animate. I think Soulstorm may be onto something in another thread. I think he's checking the current thoughts for rules-compliance.

-slapshot5