Transport Tycoon Forums

Version history:
v0.1 (for SVN trunk r12347):
- Initial release.

v0.2 (for SVN trunk r12421):
- Now you have to hold CTRL when converting rail to autoconvert depots with trains in.
- Extra options added to the autoreplacement code (thanks Bjarni!), now we only upgrade a depot if we can afford to upgrade ALL trains in depot.
- Significant code overhaul, split up CmdConvertRail into 2 functions as we now need to iterate through all selected tiles twice, to check whether we have the money necessary for the conversion first; we must do this manually because we had to cause this command to trigger 'notest' in DoCommand.
- When querying the cost of converting track, if only depots with trains are selected, gives a cost estimation of upgrading each depot (ie. max possible cost).
- When querying to cost of converting track, if any track or empty depots are selected, gives a cost estimation of just upgrading any selected track (ie. min possible cost).
- When actually doing the replacement, always tries to convert any selected track or empty depots first. If that was successful, goes through each found depot with trains in, and tries to convert it; does so if we can afford the conversion.
- The cost animation near the cursor will reflect the cost of upgrding any empty depots and track tiles (INCLUDING the track tiles of the upgraded depots that had trains in them!). Each upgraded depot will have its own cost animation too, indicating the cost of upgrading the trains and wagons in that depot.



Known bugs:
- Group handling probably isn't very good; currently we just get the EngineReplacement for the group ID of the train (ie. engV->group_id), but set it again using the ALL_GROUP group.
- We might need one or two more custom strings for this patch (although I haven't found the need for any more yet...)
- I'm told there may also be problems with NewGRF, as some GRF sets disallow certain cargo types for certain track types. As my patch basically uses the autoreplace code, I'm hoping that as long as autoreplace is maintained to work with NewGRF, so this patch will work.
- Some code cleanup is needed.
- Currently, if we're estimating the cost of autoreplacing trains in a depot, the command will fail if we don't have enough money. The command needs not to fail, but to return the cost estimation of replacement. This patch currently hacks around this by charging 100 million for replacement if the cost estimation fails, but this should be fixed (Bjarni needs to fix this one in the autoreplace code )
- Doesn't handle very well replacing wagons for NewGRF sets that might not allow certain cargo types on certain track types (what is the best way to handle this? Hmm.)
- Doesn't handle very well replacing train engines that can hold cargo - maybe (maybe not?) we should try to replace them with train engines that handle the same type of cargo.
- Doesn't have a GUI to specify replacements (do we want this? It would seem very complex, perhaps unnecessarily so.)

Bilbo wrote:First compatible? What about allowing specifying train replacement orders across railtypes (like SH-40 -> Pegasus, Asiastar -> Chimaera) and then using these for replacement instead of "first available"? This would be more clean solution IMHO

jez wrote:There is an autoreplace GUI...

libik wrote:question > I dont really understand this bug with Groups of trains... It means, that it is not able to convert more trains in one depot or it means, that we shouldnt convert more depots together?

leno4 wrote:Could someone please post there openttd.exe with the patch already installed?
Thanks

jez wrote:

libik wrote:question > I dont really understand this bug with Groups of trains... It means, that it is not able to convert more trains in one depot or it means, that we shouldnt convert more depots together?

I'm going to investigate this one a bit more. There are groups of trains, and I need to find out more but I think it's possible to group them and treat different groups differently when it comes to autoconversion. We need to handle that properly with this patch.

leno4 wrote:Could someone please post there openttd.exe with the patch already installed?
Thanks

When I have a newer version that's more robust with fewer bugs I will post such an exe.

fabca2 wrote:for thoses who say "you must extend GUI to allow replacing across rail type", I would reply :
- it's longer : you have to select each train type (imagine you have 5 differents train types...)

fabca2 wrote: - you will change every train on your network, you cannot work on a part of it (or you had to manage groups etc...)

libik wrote:how to install this patch?