Complete.Org: Mailing Lists: Archives: freeciv-dev: February 2002:
[Freeciv-Dev] Re: [PATCH] aiunit.c ai_military_findvictim() cleanup (PR#
Home

[Freeciv-Dev] Re: [PATCH] aiunit.c ai_military_findvictim() cleanup (PR#

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: Petr Baudis <pasky@xxxxxxxxxxx>
Cc: Gregory Berkolaiko <gberkolaiko@xxxxxxxxxxx>, Mike Kaufman <kaufman@xxxxxxxxxxxxxxxxxxxxxx>, freeciv-dev@xxxxxxxxxxx, bugs@xxxxxxxxxxxxxxxxxxx
Subject: [Freeciv-Dev] Re: [PATCH] aiunit.c ai_military_findvictim() cleanup (PR#1264)
From: Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx>
Date: Fri, 22 Feb 2002 21:10:49 +0100
Reply-to: rf13@xxxxxxxxxxxxxxxxxxxxxx

On Fri, Feb 22, 2002 at 08:46:09PM +0100, Petr Baudis wrote:
> Dear diary, on Fri, Feb 22, 2002 at 08:08:40PM CET, I got a letter,
> where Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx> told me, that...
> > 2) Can someone explain -2*MORT*TRADE_WEIGHTING?
> 
> We're cutting off luxury, which is part of trade, so we use TRADE_WEIGHTING.
> And MORT is just that constant meaning 'reasonably foreseeable (eek?) future'.
> And 2 is .. looks like just some constant estimated/tuned to have it working
> well. Any deeper logic in that?

2 trade to compensate one unhappiness? Related question: what is the
range of values of punit->unhappiness? [0..1] (bool) or [0..2] (as 
  ./client/gui-win32/mapview.c:  int unhappy = CLIP(0, punit->unhappiness, 2);
) suggests. Is it valid for a ruleset to set uk_happy >1?

If unhappiness isn't a bool that the formula has to be changed to
-2*MORT*TRADE_WEIGHTING*punit->unhappiness

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
 "It is not yet possible to change operating system by writing
  to /proc/sys/kernel/ostype."              sysctl(2) man page


[Prev in Thread] Current Thread [Next in Thread]