patchFreeciv - Patches: patch #3729, Attacking player specific reverse...

Show feedback again

patch #3729: Attacking player specific reverse pathfinding map for cities

Submitted by:  Marko Lindqvist <cazfi>
Submitted on:  Thu Feb 21 06:57:51 2013  
Category: aiPriority: 5 - Normal
Status: DonePrivacy: Public
Assigned to: Marko Lindqvist <cazfi>Open/Closed: Closed
Planned Release: 2.5.0Contains string changes: None

Add a New Comment (Rich MarkupRich Markup):

You are not logged in

Please log in, so followups can be emailed to you.


Wed Nov 11 22:41:57 2015, comment #2:

Later it has turned out that the main benefit of this patch is handling of movement bonus effects. Otherwise attacker with movement bonus that the defender does not have, could attack sooner than expected, in many cases taking empty city before defending player felt urgency to buy defender.

Marko Lindqvist <cazfi>
Project AdministratorIn charge of this item.
Mon Feb 25 06:47:49 2013, SVN revision 22437:

Create attacking player specific reverse path finding maps for cities
when assessing danger.

See patch #3729

(Browse SVN revision 22437)

Marko Lindqvist <cazfi>
Project AdministratorIn charge of this item.
Thu Feb 21 06:57:51 2013, original submission:

Currently assess_danger() uses single reverse pathfinding map for all the units approaching the city. This map is constructed as if all units were owned by city owner (the defender). This causes inaccuracy due to ZoC effects and restrict_infra (if enabled) being different for distinct players.

Attached patch creates reverse pathfinding map for each attacker player instead.

Marko Lindqvist <cazfi>
Project AdministratorIn charge of this item.


(Note: upload size limit is set to 1024 kB, after insertion of the required escape characters.)

Attach File(s):

Attached Files
file #17299:  AttackerMapforCity.patch added by cazfi (5kB - text/x-diff)


Depends on the following items: None found

Items that depend on this one: None found


Carbon-Copy List
  • -unavailable- added by cazfi (Submitted the item)

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.


    Error: not logged in



    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon Feb 25 06:48:01 2013cazfiStatusReady For Test=>Done
      Assigned toNone=>cazfi
    Thu Feb 21 06:57:51 2013cazfiAttached File-=>Added AttackerMapforCity.patch, #17299
    Show feedback again

    Back to the top

    Powered by Savane 3.1-cleanup