SteamedPears / 3004-SevenWonders

Seven Wonders project for 3004
1 stars 1 forks source link

More modular strategies #156

Closed Gankra closed 11 years ago

Gankra commented 11 years ago

ffffffffffffffff

spratt commented 11 years ago

I don't think we need to mix elements of strategies, just be able to pick different strategies for AI at runtime. I'll add a dropdown for each player on the host screen that lists:

Gankra commented 11 years ago

In his reqs, it seemed like it was a "bonus" requirement to be able to mix and match aspects.

spratt commented 11 years ago

Ah, gotcha. Blech.

Gankra commented 11 years ago

Hence: fffffffffffff

spratt commented 11 years ago

Christian suggested having a single dropdown for "AI type" and all the AI would be of that type. Should we do that?

Or we could just always use heuristic AI and just select which heuristics to include.

Gankra commented 11 years ago

We could have both. I can re-implement the others with the right heuristics. So the other AIs are just "pre-packaged" heuristic AIs.

spratt commented 11 years ago

Whatever's easiest on you. My seething hatred for this course leads me to believe we should do as little work as possible to still get the desired grade.

cdelahousse commented 11 years ago

Remember. Whatever gets implement will be reflected in the docs. JP wants a dynamic stub to represent a strategy, so, if we could just have one type of AI per strategy, not a composite, my life becomes easier On Apr 7, 2013 4:35 PM, "Simon David Pratt" notifications@github.com wrote:

Whatever's easiest on you. My seething hatred for this course leads me to believe we should do as little work as possible to still get the desired grade.

— Reply to this email directly or view it on GitHubhttps://github.com/SteamedPears/3004-SevenWonders/issues/156#issuecomment-16023326 .

spratt commented 11 years ago

What's makes the composite hard? I understand it's a bonus requirement.

spratt commented 11 years ago

As far as the documents are concerned, I think the action is atomically "user chooses type of AI" and everything more precise is an implementation detail, including whether you allow AI submodules or not.

Gankra commented 11 years ago

I'm more than happy to roll back the change if we don't want/care about submodules.

spratt commented 11 years ago

I'm continuing under the assumption that the difference between player choosing AI strategy and player choosing AI submodules is an implementation detail that shouldn't be reflected in the docs.

cdelahousse commented 11 years ago

Don't worry about guys. On Apr 7, 2013 8:54 PM, "Simon David Pratt" notifications@github.com wrote:

I'm continuing under the assumption that the difference between player choosing AI strategy and player choosing AI submodules is an implementation detail that shouldn't be reflected in the docs.

— Reply to this email directly or view it on GitHubhttps://github.com/SteamedPears/3004-SevenWonders/issues/156#issuecomment-16028129 .

Gankra commented 11 years ago

So what are we (me) doing here?

spratt commented 11 years ago

Christian said in a text to go ahead with the submodules, and we'd talk about docs tomorrow.

Gankra commented 11 years ago

Resolved?