Topic: Solution proposals for maritime shipping problems
Nordfriese![]() |
Posted at:
2020-02-05, 20:13 UTC+1.0
Had a quick look at the code. When the economy doesn't have enough tools it creates request in warehouses for these tools. The tools are then brought to the requesting warehouses. Warehouses are iterated in order of their creation, this should be fixed. (Code is in Edit: This is not trivial, because the economy batch-creates as many workers of a type as needed and possible
Edited:
2020-02-05, 20:15 UTC+1.0
![]() ![]() |
JanO![]() |
Posted at:
2020-02-05, 23:34 UTC+1.0
Question (king): ![]() ![]() |
hessenfarmer![]() |
Posted at:
2020-02-06, 10:36 UTC+1.0
If this would be possible I would rate it as a bug as Colonies connected by ports should be in the same economy. ![]() ![]() |
JanO![]() |
Posted at:
2020-02-06, 12:43 UTC+1.0
That's why I ask. I know that once I could set the demands on different islands individually, but I don't remember if I had ships there (should so, otherwise I would not have had settlements on different islands). But that was probably b18 or something. ![]() ![]() |
GunChleoc![]() |
Posted at:
2020-02-21, 20:06 UTC+1.0
Yes, there is: https://www.widelands.org/news/2019/May/4/widelands-build-20-released/ Busy indexing nil values ![]() ![]() |
king_of_nowhere![]() |
Posted at:
2020-02-22, 00:47 UTC+1.0
damnit! i was sure i had answered to this, but now i see i didn't. sorry for taking too long. y, the colonies are like you describe. and yes, now that i know that a requested worker generates a request for the tool in the oldest warehouse, i know i could have fixed it by increasing the demand from the stock. in that case, the tool would have been sent to the closer warehouse, and would have been used there to make the worker. ![]() ![]() |
Solstice_s_Return![]() Topic Opener |
Posted at:
2020-02-22, 10:55 UTC+1.0
Yeah, I'm aware of this now as I ve played a bit with b20-1 too and recently took a quick look into 20-7 also. ![]() ![]() |
Nordfriese![]() |
Posted at:
2020-04-23, 20:11 UTC+2.0
There' s a pull request up to refactor the shipping algorithm in a way that should fix all issues described here as well as a bunch of hangs, crashes, and asserts: https://github.com/widelands/widelands/pull/3857 There are still a few asserts left in this branch, but I'd appreciate testing and reporting of unexpected/undesired/suboptimal behaviour For testers who like to experiment – you can try playing around with the weighting constants in The logic of the algorithm is documented in the same file, lines 330-357 and subsequent comments ("1st-6th pass"). ![]() ![]() |