Page 8 of 12
Re: [Patch] Daylength, Cargodist with sprinkles
Posted: 11 Apr 2010 11:45
by ostlandr
Thanks for the binary SpComb!
Has anybody else noticed this behavior:
When setting up a new cargo route with a transfer, a vehicle will pick up a load of "to any station" cargo. But since it never arrives at a station that accepts the cargo, it drives around full until you order it to unload at the transfer station. Once the LinkGraph kicks in and the cargo has a destination, you can cancel the forced unload. Just a minor annoyance, but I wonder if I'm doing something wrong here.
Re: [Patch] Daylength, Cargodist with sprinkles
Posted: 11 Apr 2010 12:17
by bluebottle
Obsolete (already!): check
here
This is a build of r19596 for Ubuntu 9.10 Linux (Karmic Koala), using SpComb's patch above. Check that post for information on the patches included as well as any caveats.
- As with the distro OpenTTD package, you need to have the data files in /usr/share/games/openttd/data/.
- This build is unstripped to provide better (i.e. useful) core dumps/backtraces.
- Any warnings about downgrading to version 1.1_svn_0 can be safely ignored.
Download: openttd_cargodist_sprinkles-r19596M-no_debug-unstripped-i386.deb
Crash reports / feedback:
Please report
any and all crashes from this build in
this thread, unless you can also reproduce them in a nightly or cargodist/autosep build.
It's a good idea to also post the
crash.* files that OpenTTD produces, along with any relevant information relating to the crash (such as a recent savegame and a description of the circumstances of the crash).
Additionally, comments or suggestions as to the behavior of the auto-separation patch should go into the
Automated timetables and separation topic.
Re: [Patch] Daylength, Cargodist with sprinkles
Posted: 11 Apr 2010 14:04
by DJ Nekkid
apparently wont the new version load savegames from the old version... so the server will have a new game

Will be online soon
edit:
Online

Re: [Patch] Daylength, Cargodist with sprinkles
Posted: 11 Apr 2010 14:36
by Fenris
Hello
At first, thanks for your work and please excuse my bad english. Hope I can explain my problem anyway.
I play this patch with FIRS (v1.2 at the moment) and since the binary from version 19527 I have a problem with the engineering parts cargo. The new binary shows the same behavior.
As soon as I try to load a savegame with an established route with the engineering parts cargotyp the game crashes, but only if I load from the main menu. As long as I load it from a running game, the crash doesn't happen. I’ve tested this a few times with different saves, but I’m not really sure, it will never happen. Also routes with ore, coal or steel don't make problems.
I tried the same with the official version 1.0 and also with chills patchpak. Both work fine. I even can load a save with an established cargoroute from chills patchpak (v19525) with the sprinkles patch version without any problem.
The crash.log shows the message: NOT_REACHED triggered at line 148 of ..\src\linkgraph\demands.cpp but when the crash occurs windows shows message gamepath\data\opengfx1_base.rgf.
Re: [Patch] Daylength, Cargodist with sprinkles
Posted: 11 Apr 2010 14:55
by fonso
Ouch, this suggests that I fail to properly clear the link graphs when loading a game. Can I get the savegame you're trying to load? Was this a game created with a version of cargodist(+sprinkles) or a game from trunk?
Edit 1: I just found out that the save/load code is still broken in the first case ... a fix will soon be available.
Edit 2: Fixed this one. At the moment when the link graph jobs were started after loading a savegame the cargo classes from the previous game were used. This could lead to link graphs operating on manually distributed cargo which is forbidden. Now they are started after loading the cargo classes. In principle you can still create a savegame with a link graph job operating on cargo with manual distribution (thus desyncs and crashes). But that requires fiddling with the cargo newgrfs while the game is running. I think I don't have to support that.
Re: [Patch] Daylength, Cargodist with sprinkles
Posted: 11 Apr 2010 16:10
by SpComb
⬆ See Above ⬆
Git repo: http://git.qmsk.net/?p=openttd
Download: openttd-cargodist-sprinkles-r19601M-win32.zip (
.pdb.gz,
.patch,
.svn.patch,
.log).
The win32 .zip and git .patch are also attached.
Savegame compatibility:
This build should be able to load games saved by all previous official OpenTTD versions, as well as the matching vanilla cargodist build.
With a bit of luck, this build might be able to load games from the previous build. This quirk will be rectified in future builds
Crash reports / feedback:
Please report
any and all crashes from this build in
this thread, unless you can also reproduce them in a nightly or cargodist/autosep build.
It's a good idea to also post the
crash.* files that OpenTTD produces, along with any relevant information relating to the crash (such as a recent savegame and a description of the circumstances of the crash).
Additionally, comments or suggestions as to the behavior of the auto-separation patch should go into the
Automated timetables and separation topic.
Re: [Patch] Daylength, Cargodist with sprinkles
Posted: 11 Apr 2010 17:39
by DJ Nekkid
ill test in a few mins, just need to finish watching a footballmatch
edit:
server up and running, r19601M, loaded the last game

Re: [Patch] Daylength, Cargodist with sprinkles
Posted: 11 Apr 2010 18:02
by Fenris
It seems my english was good enough to tell what i want.
For me it works now. Thanks for the quick fix and keep up the good work.

Re: [Patch] Daylength, Cargodist with sprinkles
Posted: 13 Apr 2010 00:18
by bluebottle
Obsolete, check
here.
This is a build of r19601 for Ubuntu 9.10 Linux (Karmic Koala), using SpComb's patch above. Check that post for information on the patches included as well as any caveats.
- As with the distro OpenTTD package, you need to have the data files in /usr/share/games/openttd/data/.
- This build is unstripped to provide better (i.e. useful) core dumps/backtraces.
- Any warnings about downgrading to version 1.1_svn_0 can be safely ignored.
Download: openttd_cargodist_sprinkles-r19601M-no_debug-unstripped-i386.deb
Crash reports / feedback:
Please report
any and all crashes from this build in
this thread, unless you can also reproduce them in a nightly or cargodist/autosep build.
It's a good idea to also post the
crash.* files that OpenTTD produces, along with any relevant information relating to the crash (such as a recent savegame and a description of the circumstances of the crash).
Additionally, comments or suggestions as to the behavior of the auto-separation patch should go into the
Automated timetables and separation topic.
Re: [Patch] Daylength, Cargodist with sprinkles
Posted: 13 Apr 2010 17:47
by charlymorton
I'm unable to load this savegame which was saved on [Patch] Daylength, Cargodist with sprinkles r19527M
Edit: Just to specify. I'm unable to load it with the patch r19601M. Get an invalid chunk size error.
Re: [Patch] Daylength, Cargodist with sprinkles
Posted: 13 Apr 2010 23:30
by SpComb
charlymorton wrote:I'm unable to load this savegame which was saved on [Patch] Daylength, Cargodist with sprinkles r19527M
Edit: Just to specify. I'm unable to load it with the patch r19601M. Get an invalid chunk size error.
Read carefully:
SpComb wrote:Savegame compatibility:
This build should be able to load games saved by all previous official OpenTTD versions, as well as the matching vanilla cargodist build.
With a bit of luck, this build might be able to load games from the previous build (singular).
In other words, known behaviour. You'd have to convince fonso to start maintaing some kind of backwards-compatibility for saveload across cargodist updates, but it might not be quite there yet.
Until then, the occasional update might not break the saveload format, but there are exactly zero guarantees across the builds here. It sucks, I know, but it's more difficult to achieve than one might think.
Re: [Patch] Daylength, Cargodist with sprinkles
Posted: 14 Apr 2010 08:24
by charlymorton
Thanks.
Yes, maybe I should read more carefully.

Re: [Patch] Daylength, Cargodist with sprinkles
Posted: 20 Apr 2010 18:58
by aantono
ostlandr wrote:Thanks for the binary SpComb!
Has anybody else noticed this behavior:
When setting up a new cargo route with a transfer, a vehicle will pick up a load of "to any station" cargo. But since it never arrives at a station that accepts the cargo, it drives around full until you order it to unload at the transfer station. Once the LinkGraph kicks in and the cargo has a destination, you can cancel the forced unload. Just a minor annoyance, but I wonder if I'm doing something wrong here.
Yep, I'm getting the same behavior. It is the most notoriously visible if setting up a feeder to the airport that is outside of the city limits and on its own does not accept and supplies cargo (pax and mail). It is especially most difficult early on in the game (as in 1920-30) when the speed of the airplanes is very low, and it takes a very long time for the LinkGraph to kick in and keep the route stable.
Any suggestions on the best course of action on how to deal with this situation most effectively?
Re: [Patch] Daylength, Cargodist with sprinkles
Posted: 20 Apr 2010 22:11
by fonso
aantono wrote:Any suggestions on the best course of action on how to deal with this situation most effectively?
1. Watch the links in the smallmap and check if they're breaking down. If they are, use more vehicles. After all, you're also losing passengers if you keep them waiting very long between the vehicles' arrivals.
2. Decrease linkgraph recalculation interval to have the link graph calculation run more often. Probably the default 16 days are OK, but especially in the early game you might have lots of small isolated components in which case they aren't.
Re: [Patch] Daylength, Cargodist with sprinkles
Posted: 23 Apr 2010 19:49
by Kogut
Re: [Patch] Daylength, Cargodist with sprinkles
Posted: 23 Apr 2010 22:31
by SpComb
It'll find its way into trunk sooner than later, and it doesn't need any special long-term intensive testing.
I'm rather tending towards minimizing patch flux for a small number of significant patches.
Re: [Patch] Daylength, Cargodist with sprinkles
Posted: 27 Apr 2010 09:06
by ReisRyos
It is possible new build, please?

Re: [Patch] Daylength, Cargodist with sprinkles
Posted: 27 Apr 2010 12:41
by SpComb
ReisRyos wrote:It is possible new build, please?

It would help to say what it is that needs updating, but I guess there's been all kinds of useful fixes in trunk over the past two weeks that are worth including, as well as the symmetric demands tweaks.
I'll update today or tomorrow, particularly if fonso synchs..
Re: [Patch] Daylength, Cargodist with sprinkles
Posted: 27 Apr 2010 18:30
by fonso
SpComb wrote:I'll update today or tomorrow, particularly if fonso synchs..
There you go. Fresh version with all the documentation I've written over the weekend and merged with trunk.
Re: [Patch] Daylength, Cargodist with sprinkles
Posted: 28 Apr 2010 20:32
by SpComb
Bump along trunk, some minor
cargodist tweaks, and a bunch of useful stuff from trunk, leading up towards 1.0.1.
Download: openttd-cargodist-sprinkles-r19733M-win32.zip (
.pdb.gz,
.patch,
.svn.patch,
.log).
The git .patch is also attached, but the win32 .zip doesn't fit in the 4MiB attachment size limit anymore. Download from the above link.
Savegame compatibility:
This build should be able to load games saved by all previous official OpenTTD versions, as well as the matching vanilla cargodist build.
With a bit of luck, this build might be able to load games from
the previous build. This quirk will hopefully be rectified in future builds.
Crash reports / feedback:
Please report
any and all crashes from this build in
this thread, unless you can also reproduce them in a nightly or cargodist/autosep build.
It's a good idea to also post the
crash.* files that OpenTTD produces, along with any relevant information relating to the crash (such as a recent savegame and a description of the circumstances of the crash).
Additionally, comments or suggestions as to the behavior of the auto-separation patch should go into the
Automated timetables and separation topic.