[wesnoth-commits] [wesnoth/wesnoth] 8fc6ca: Experimental AI: fix village_hunt CA bug

[mattsc]

  Branch: refs/heads/1.14
  Home:   https://github.com/wesnoth/wesnoth
  Commit: 8fc6cabf7d5028b3e332fcdacf8a4fde1c555c44
      https://github.com/wesnoth/wesnoth/commit/8fc6cabf7d5028b3e332fcdacf8a4fde1c555c44
  Author: mattsc <mattsc000 at gmail.com>
  Date:   2020-01-01 (Wed, 01 Jan 2020)

  Changed paths:
    M data/ai/lua/generic_rush_engine.lua

  Log Message:
  -----------
  Experimental AI: fix village_hunt CA bug

This is really just a work around for a CA that is very much sub-par though.  I will remove this CA from the ExpAI next.


  Commit: 450e3d1a489f3b47ea5bf50aa88ea5e36660fc52
      https://github.com/wesnoth/wesnoth/commit/450e3d1a489f3b47ea5bf50aa88ea5e36660fc52
  Author: mattsc <mattsc000 at gmail.com>
  Date:   2020-01-01 (Wed, 01 Jan 2020)

  Changed paths:
    M data/ai/lua/generic_rush_engine.lua
    M data/core/macros/ai.cfg

  Log Message:
  -----------
  Experimental AI: do not use village hunt CA

The algorithm used in this CA is too simple to work reliably in a general setting, it tends to send whole groups of units toward small numbers of villages, or even individual ones. In its current version, it should not be used at all, not even in the Experimental AI. The recommended way to emphasize village hunting is to set the village_value aspect to a larger-than-default value and let the move-to-targets CA take care of it.

An updated version of this CA's code is, however, left in place for potential future work as an external CA in Wesnoth 1.15.


Compare: https://github.com/wesnoth/wesnoth/compare/0ea6dcceb2fd...450e3d1a489f