[Freeciv-Dev] [patch #5201] pf_reverse_map() improvements
Follow-up Comment #6, patch #5201 (project freeciv): New version: * do not consider the unit moves left. Assume the units always have their whole move rate, because punit-moves_left is not restored yes when using pf_reverse_map. (file #22205) ___ Additional Item Attachment: File name: pf_reverse_map5.patch Size:23 KB ___ Reply to this item at: http://gna.org/patch/?5201 ___ Message posté via/par Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #5201] pf_reverse_map() improvements
Follow-up Comment #5, patch #5201 (project freeciv): New version: * do not consider parameter-utype-unknown_move_cost when omniscience is set ; * consider any move to the target tile as an action, to consider all units (notably the ones not able to take over but able to fight, or to simulate overlap mode for transports). (file #22168) ___ Additional Item Attachment: File name: pf_reverse_map4.patch Size:22 KB ___ Reply to this item at: http://gna.org/patch/?5201 ___ Message posté via/par Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #5201] pf_reverse_map() improvements
Update of patch #5201 (project freeciv): Status: Need Info = Ready For Test ___ Follow-up Comment #4: I think you're basically reverting an improvement made some time ago. Unless something else has changed also, this map is not only for map knowledge, but for any unit movement restrictions. That includes movement bonuses (Lighthouse etc) and ZoC (attacking units of the same owner are not preventing each other from moving) Okay. Agreed (except that pf_reverse_map never used ZoC restrictions, because ZoC is too flexible due to unit moves, and movement bonuses are already handled in unit_move_rate()). However, the map knowledge is still wrong. But because assess_danger() already take care of units the city owner cannot see, maybe omniscient should be always set to TRUE in this case. This would also remove the usage of unknown_move_cost. (file #22157) ___ Additional Item Attachment: File name: pf_reverse_map3.patch Size:20 KB ___ Reply to this item at: http://gna.org/patch/?5201 ___ Message posté via/par Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #5201] pf_reverse_map() improvements
Follow-up Comment #3, patch #5201 (project freeciv): I think you're basically reverting an improvement made some time ago. Unless something else has changed also, this map is not only for map knowledge, but for any unit movement restrictions. That includes movement bonuses (Lighthouse etc) and ZoC (attacking units of the same owner are not preventing each other from moving) ___ Reply to this item at: http://gna.org/patch/?5201 ___ 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 #5201] pf_reverse_map() improvements
Update of patch #5201 (project freeciv): Status: Ready For Test = In Progress ___ Reply to this item at: http://gna.org/patch/?5201 ___ Message posté via/par Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #5201] pf_reverse_map() improvements
URL: http://gna.org/patch/?5201 Summary: pf_reverse_map() improvements Project: Freeciv Submitted by: pepeto Submitted on: dim. 07 sept. 2014 13:33:30 CEST Category: ai Priority: 5 - Normal Status: Ready For Test Privacy: Public Assigned to: pepeto Originator Email: Open/Closed: Open Discussion Lock: Any Planned Release: 2.6.0 ___ Details: * use pf normal maps, ignoring units, cities, ZOC, refuel points etc; * remove the ability to make paths (anyway they weren't really usable); * record only a pf_position instead of a pf_map in the hash; * assess_danger(): use a single reverse map. The player pointer should be the city owner, because it is only used to know if the map is known or not. ___ File Attachments: --- Date: dim. 07 sept. 2014 13:33:30 CEST Name: pf_reverse_map2.patch Size: 24 ko By: pepeto http://gna.org/patch/download.php?file_id=22094 ___ Reply to this item at: http://gna.org/patch/?5201 ___ Message posté via/par Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
