[Freeciv-Dev] Re: (PR#9866) S1_14: AI fixes

["Per I. Mathisen" <per@xxxxxxxxxxx>]

<URL: http://rt.freeciv.org/Ticket/Display.html?id=9866 >

On Sun, 29 Aug 2004, Gregory Berkolaiko wrote:
> Your latest patch in the "fun Ai game" thread does much better job: it
> retains the check for whether we got assigned to the city and it compares
> with defense rather than 0.
>
> So I suggest we use it to fix S-branch.

Ok. I will do some more testing first, though.

> It does seem to contain some noise with "units" variable.

It is not noise...

+  /* Treat grave danger */
+  if (pcity->ai.grave_danger > units) {

... it is a way to treat grave danger. Although you might think in cases
of "grave danger" we should all stay, the AI is way too generous setting
the grave danger flag, so we can't do that. But we need to take it
somewhat into consideration, and use the fact that it is nearly always a
very low number when we are not in really grave danger.

> Also, can you try and feed the AI tickets through RT please. You will
> have to use RT interface to create the ticket in the right queue, but it's
> worth it. Apart from obvious benefits (like instant archiving), I can
> download patches from RT rather than save them at my mail server and then
> ftp it around -- much bother.

Ok.

  - Per