search

tomjn / Shard

A reprogrammable AI framework for the Spring Engine
https://shard.tomjn.com
17 stars 13 forks source link

add controlpoints and map drawing functions to API #101

Closed eronoobos closed 8 years ago

eronoobos commented 8 years ago

@Tarendai not merging yet because it concerns the API and you might want to take a look

tomjn commented 8 years ago

The naming is my main concern here, as we already have capturing in the API ( or should have it ), for capturing other units OTA style. Perhaps the API calls need to be more descriptive. I'd also like a check in the API that can be called telling the AI if these are relevant or not, e.g. game:usesCapturePoints or something to that effect

Otherwise, looks good

eronoobos commented 8 years ago

True, perhaps ControlPointNonCapturingUnits and ControlPointCaptureRadius. And CapturerBehaviour may not be the best name either.

I'm not sure how to get the game to tell us if the points are relevant to it. That check was my intention with map:AreControlPoints() but of course that just tells us if there are any points at all. Maybe for now I'll make game:usesCapturePoints() an alias of that, because currently it's not a bad assumption--only games that have this particular control point gadget will have control points.