Author Topic: Another look at Geoscape AI.  (Read 17107 times)

Offline karvanit

  • Captain
  • ***
  • Posts: 94
    • View Profile
Re: Another look at Geoscape AI.
« Reply #30 on: November 08, 2012, 06:57:48 pm »
First of all, thanks for the clean pull request!

so i finally get github "figured out", sit down to take a look at your code, and you've used completely different methods in ruleAlienMission than in... every other ruleset in openxcom.
i'm at a loss here. i just don't know what to do.
should i rewrite it? should i try to figure out wtf you're doing and try to do it that way?

Besides the I/O (which I have now brought more in line with the rest of OpenXcom) the RuleAlienMission only has two functions, one to get the mission type and one to select a race. These are the only mission parts I have in the ruleset so far. I have hidden the 'chose a race from among the possibilities' behind a separate class that can be used to select based on weights, because this is common functionality (choose mission type based on region, choose region for mission, all these are the same).

Volutar

  • Guest
Re: Another look at Geoscape AI.
« Reply #31 on: November 09, 2012, 04:13:19 am »
Quote
Is it just by position (offset) in the table?
Sure. It's the only way xcom adresses any data.