bvschaik / julius

An open source re-implementation of Caesar III
GNU Affero General Public License v3.0
2.84k stars 317 forks source link

[BUG] Warehouse being cleared #699

Closed leobastiani closed 1 year ago

leobastiani commented 1 year ago

Seems to be a bug that my Warehouse start to be emptied just after configuring Granaries

https://github.com/bvschaik/julius/assets/8663061/28aa23a6-1d53-4aa0-bb2d-90a5e695224a

autosave.zip

Some details about my Caesar 3 assets file I have the .iso from the original game in Portuguese Brazil, so I'm not using Steam either GOG version Another strange piece of information about my folder's game, I'm use git with nodemon to watch for changes on autosave.sav and keep a history of autosave, Sometimes, months in game passes too quick and I already saw an autosave corrupted, I'm not sure if it's related to the fact I'm using git

crudelios commented 1 year ago

So from what I understand, if you set a warehouse to "empty" and then no granaries accept that resource, the granary will fully empty by itself without delivering the resource anywhere, is that right?

Regarding your corrupted autosave, it's very possible the game is saving over while also you're syncing the file. That would certainly result in corruption. There's really no way around that.

leobastiani commented 1 year ago

So from what I understand, if you set a warehouse to "empty" and then no granaries accept that resource, the granary will fully empty by itself without delivering the resource anywhere, is that right?

The warehouse is not set to empty warehouse You can see someone stealing my wheat haha, and flashing between seconds 31 and 35 on the video warehouse-flashing-bug

bvschaik commented 1 year ago

As discussed on Discord after investigation by @crudelios: the wheat warehouse is trying to deliver to a warehouse that has two road connections. Since the road access tile of the target warehouse is unreachable, the deliveryman is lost, taking the wheat with him. This works the same in Caesar 3. Since fixing this requires reworking the pathing algorithm, breaking compatibility with C3 along the way, we won't fix this.

crudelios commented 1 year ago

Also, this issue is somewhat fixed in Augustus as the cartpusher will in certain cases go offroad, allowing access to the main warehouse tile.