[Freeciv-Dev] [bug #20504] City defenders rampaging

2013-02-15 Thread Marko Lindqvist
Update of bug #20504 (project freeciv):

  Status:  Ready For Test = Fixed  
 Assigned to:None = cazfi  
 Open/Closed:Open = Closed 


___

Reply to this item at:

  http://gna.org/bugs/?20504

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


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


[Freeciv-Dev] [bug #20504] City defenders rampaging

2013-02-12 Thread Marko Lindqvist
URL:
  http://gna.org/bugs/?20504

 Summary: City defenders rampaging
 Project: Freeciv
Submitted by: cazfi
Submitted on: Wed 13 Feb 2013 05:34:25 AM EET
Category: ai
Severity: 3 - Normal
Priority: 5 - Normal
  Status: None
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Release: 
 Discussion Lock: Any
Operating System: None
 Planned Release: 

___

Details:

dai_military_defend() is supposed to handle unit assigned to defend a city,
including the case unit already is in that city and enemy is next to it.
dai_military_defend() calls dai_military_rampage() with parameters that denote
that any target is worth going after. This may mean that defender leaves the
city to kill target a couple of tiles away. So, if there's strong attacker
adjacent to city ready to enter, and weaker enemy further away, defender
leaves city defenseless for the strong attacker to take while it itself hunts
down the weaker enemy.

It also seems to me that even if there was no stronger attacker present, the
code would never draw the defender back to city. Once it's outside, it next
turn loses its DEFEND_HOME task and only units already inside city get that
task (re)assigned.




___

Reply to this item at:

  http://gna.org/bugs/?20504

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


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


[Freeciv-Dev] [bug #20504] City defenders rampaging

2013-02-12 Thread Marko Lindqvist
Update of bug #20504 (project freeciv):

  Status:None = Ready For Test 

___

Follow-up Comment #1:

Here's the patch I'm testing.
- Leave city only if you can take empty enemy city - I assume this to be
statistically worth the risk that enemy then takes the original city
- Attack adjacent tiles (staying itself in the city) for juicy targets - this
might mean killing entire stack of enemies that would otherwise get to attack
against us each and every

(file #17186)
___

Additional Item Attachment:

File name: DefenderRampage.patch  Size:1 KB


___

Reply to this item at:

  http://gna.org/bugs/?20504

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


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


[Freeciv-Dev] [bug #20504] City defenders rampaging

2013-02-12 Thread Marko Lindqvist
Update of bug #20504 (project freeciv):

 Planned Release: = 2.4.0, 2.5.0   

___

Follow-up Comment #2:

 Attack adjacent tiles (staying itself in the city) for juicy
 targets

Above works in this new version. Also S2_4 version included.

(file #17189, file #17190)
___

Additional Item Attachment:

File name: DefenderRampage-2.patchSize:1 KB
File name: DefenderRampage-S2_4-2.patch   Size:0 KB


___

Reply to this item at:

  http://gna.org/bugs/?20504

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


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