I see you had posted new map (nice one) and I hope you tested this branch on this map and found it better than current implementation in trunk...
I see two border scenarios 1. Multiple islands with one port on each of them 2. One land = one sea - with multiple ports on them (here I am not sure if you think about this) The logic must work for both and all between. I have not tested this branch, but on the other side I would welcome if somebody worked on the seafaring from AI point of view. Especially if you intend to go on improving this. And then next release can be very far so this is not last time seafaring is touched I believe. So to summarize - if you tested it and are sure this is improvement - let it go... -- https://code.launchpad.net/~7010622-q/widelands/topple-seafaring-1/+merge/310436 Your team Widelands Developers is requested to review the proposed merge of lp:~7010622-q/widelands/topple-seafaring-1 into lp:widelands. _______________________________________________ Mailing list: https://launchpad.net/~widelands-dev Post to : [email protected] Unsubscribe : https://launchpad.net/~widelands-dev More help : https://help.launchpad.net/ListHelp
