URL:
  <http://gna.org/patch/?2885>

                 Summary: Fix autosettler to use rail connectivity bonus when
planning for railroads
                 Project: Freeciv
            Submitted by: None
            Submitted on: Wed 10 Aug 2011 04:28:51 AM UTC
                Category: ai
                Priority: 5 - Normal
                  Status: None
                 Privacy: Public
             Assigned to: None
        Originator Email: 
             Open/Closed: Open
         Discussion Lock: Any
         Planned Release: 

    _______________________________________________________

Details:

When thinking about building a road, autosettlers also consider building the
road for the sole purpose of later building a railroad.  Both road and rail
building are subject to a connectivity bonus, so that autosettlers prefers to
build roads/rail on tiles that help connect the nation even when the
(rail)road otherwise has no benefit (e.g. to trade).

The code presently uses the road connectivity bonus (which favors building
regular roads on certain tiles adjacent to other regular roads) even when
building the road strictly as a prerequisite for building a railroad.  It
seems more appropriate, however, to use the rail connectivity bonus (which is
similar, but for railroads) in this situation.

This patch changes the code to use the rail connectivity bonus in this
situation and also cleans up the code a bit to remove a confusing (and
incorrectly named) duplicated variable.

TODO: The road/rail connectivity bonus computations need to be revisited. 
Their logic is difficult to follow and seems a bit buggy, plus the bonus may
not be enough (autosettlers will mine hills for ages before putting much
effort into roads, which can be much more useful to an early nation). 
Regardless, this patch will pave the way for more precise future adjustments.



    _______________________________________________________

File Attachments:


-------------------------------------------------------
Date: Wed 10 Aug 2011 04:28:51 AM UTC  Name:
fix-autosettler-road-to-rail-extra.patch  Size: 2kB   By: None

<http://gna.org/patch/download.php?file_id=13818>

    _______________________________________________________

Reply to this item at:

  <http://gna.org/patch/?2885>

_______________________________________________
  Message sent via/by Gna!
  http://gna.org/


_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev

Reply via email to