Those are some interesting observations. Thanks for the feedback.
Zephyris wrote:
[*] Non-stop only orders really doesn't make sense to me... Is there really no way to get around this? You could, for example, only use stations in the orders list for generating the deterministic passengers and other stations the vehicle happens to stop at could add non-deterministic passengers...
Any way around this either involves ridiculous amount of work or is incredibly fragile. For every scheme of finding the in-between stations of an order I could come with up so far, I could easily construct an example where it totally fails. So I'd say either we keep it as it, or we force all orders to be non-stop when cargodist is switched on, or we just disregard the nondeterminism and load passengers for the next stop in the order list. Those can all be done in a matter of minutes. I don't care about that as I don't think there's too many people using that "stopping" feature with cargodist anyway. I've kept the current behaviour as it enabless that trick of forcing a vehicle to load all available cargo by placing a stopping order after the one where it should load. But that's a hack anyway.
Zephyris wrote:
[*] The user interface (on the minimap) seems far too confusing, there is just too much information being shown. Before this patch is suitable for "general consumption" I think you seriously need to think about what information needs to be shown. I would say you need:
-- Link graphs for different cargos. Only one cargo should be shown at once to simplify the view.
-- Capacity of each link (C)
-- Usage of each link (U)
-- Spare capacity of each link (either C-U or C/U)
I would strongly suggest getting rid of the bar charts and numbers and replacing them with colour coding for the link. If you are happy to only display one cargo at once then at any time you would only be showing link capacity OR link usage OR spare capacity. These values could easily be mapped to a green to red scale... This may well also solve the requirement for the minimap zoom. In general I do find the minimap zoom useful though...
Well, the smallmap link stats display is a big fail. At the same time it has been the single greatest time sink of all the branches I'm maintaining. I guess I'll have to get myself to go back to the drawing board there ...
Zephyris wrote:
Other little bugs and comments:
[*] Calculation mouse position on the minimap for mouse-over information breaks if the title bar is not the default size (try this grf to see this bug
http://www.tt-forums.net/viewtopic.php?f=26&t=50095 ). It looks like you are calculating the mouse position/link position from the top right corner of the window not the small map view itself.
Oops. Actually it didn't even work properly for the normal size. It's fixed now.
Zephyris wrote:
[*] The buttons to select the cargo need to look more like buttons - try lifting the ones from the cargo income graph. Similarly the other graph display options don't really look like buttons.
I'll go with Eddi here. As long as it's acceptable for the industry map it's also acceptable for the link stats map.
The guy on the picture is not me, it's Alonso.