← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~7010622-q/widelands/topple-seafaring-1 into lp:widelands

 

I tested my proposed logic on a "one-world" island and it turned out the expedition found portspaces in secondary access although it doesn't per-se perform secondary island circling. This is because the map is a torus and the ship is reflected only onto the island again under various angles and situations. So it works!

Regarding your considerations, however, I would suggest a repetition probability of 25% (instead of 0). In this manner we could safely implement the improvements implied and work on more profound revisions later on. Would that find your agreement?

-- 
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.


References