[Freeciv-Dev] [patch #3177] Autosettler to consider other units doing ACTIVITY_GEN_ROAD as road builders

2012-03-25 Thread Marko Lindqvist
Update of patch #3177 (project freeciv):

  Status: In Progress = Ready For Test 


___

Reply to this item at:

  http://gna.org/patch/?3177

___
  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] [patch #3177] Autosettler to consider other units doing ACTIVITY_GEN_ROAD as road builders

2012-03-21 Thread Marko Lindqvist
Update of patch #3177 (project freeciv):

  Status:  Ready For Test = In Progress

___

Follow-up Comment #4:

Oh, how old code sort-of-works by luck. That
consider-normal-road-being-worked-on as Railroad triggers Railroad network
building at some point (AI doesn't want to start rail network building, but it
wants to expand one it thinks it already has.) On the other hand, lack of
consider-exististing-roads-as-rails means it knows that there's no alternative
route already, but it has to build rails now.

Problem with my previous version of this patch is that by considering existing
roads as potential rails, it causes AI to often thinks that it doesn't need to
build rails where it currently is, as there's alternative route.

___

Reply to this item at:

  http://gna.org/patch/?3177

___
  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] [patch #3177] Autosettler to consider other units doing ACTIVITY_GEN_ROAD as road builders

2012-03-18 Thread Marko Lindqvist
Follow-up Comment #2, patch #3177 (project freeciv):

Split old behavior related things to bug #19560.

I don't think I should make new code to provide 100% identical autosettler
behavior to old one, but am testing newly written version.

___

Reply to this item at:

  http://gna.org/patch/?3177

___
  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] [patch #3177] Autosettler to consider other units doing ACTIVITY_GEN_ROAD as road builders

2012-03-18 Thread Marko Lindqvist
Update of patch #3177 (project freeciv):

  Status: In Progress = Ready For Test 

___

Follow-up Comment #3:

- Consider any adjacent tile with any road, either existing or being worked
on, that target road depends on, to have the target road eventually.

(file #15378)
___

Additional Item Attachment:

File name: ASGenRoadOtherSettler-2.diff   Size:5 KB


___

Reply to this item at:

  http://gna.org/patch/?3177

___
  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] [patch #3177] Autosettler to consider other units doing ACTIVITY_GEN_ROAD as road builders

2012-03-17 Thread Marko Lindqvist
Update of patch #3177 (project freeciv):

Category: general = ai 
  Status:  Ready For Test = In Progress

___

Follow-up Comment #1:

 I don't know if old behavior was a bug or intentional trick
...

Either way, changing that caused AI not to build railroads at all. Have to
think what to do.

___

Reply to this item at:

  http://gna.org/patch/?3177

___
  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] [patch #3177] Autosettler to consider other units doing ACTIVITY_GEN_ROAD as road builders

2012-03-01 Thread Marko Lindqvist
URL:
  http://gna.org/patch/?3177

 Summary: Autosettler to consider other units doing
ACTIVITY_GEN_ROAD as road builders
 Project: Freeciv
Submitted by: cazfi
Submitted on: Fri 02 Mar 2012 01:11:11 AM EET
Category: general
Priority: 5 - Normal
  Status: Ready For Test
 Privacy: Public
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 2.5.0

___

Details:

Autosettler function road_bonus() wants to know if some other unit is
currently building road to nearby tiles. Goal of this patch was to make it
understand that units doing ACTIVITY_GEN_ROAD with correct road type as target
are doing just that. Function now takes target road type as parameter, and is
generally gen-roads ready.

I don't know if old behavior was a bug or intentional trick, but while
evaluating railroad connectivity benefits nearby roads were considered
railroads. Maybe it was just to get railroad building started by claiming that
first railroad tile already would connect existing railroads. And one can
consider it a predicion rather than statement of current situation; road is
going to be updated to railroad if needed.



___

File Attachments:


---
Date: Fri 02 Mar 2012 01:11:11 AM EET  Name: ASGenRoadOtherSettler.diff  Size:
3kB   By: cazfi

http://gna.org/patch/download.php?file_id=15318

___

Reply to this item at:

  http://gna.org/patch/?3177

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


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