search

cff29546 / pzmap2dzi

A command-line tool to convert Project Zomboid map data into Deep Zoom format
MIT License
52 stars 17 forks source link

Many missing tile warning. missing interiors and tiles. #4

Closed verdra closed 1 year ago

verdra commented 1 year ago

when running i receive a lot of missing tile warning during the base and base_top steps. after the process completes the map has a number of black regions in both top and isometric. the black regions are primarily in interiors, water, and field tiles.

there is also missing images for some chunks of the most zoomed out regions.

missing tile: blends_street_01_85 missing tile: blends_street_01_80 missing tile: blends_street_01_87 missing tile: blends_street_01_80 missing tile: blends_street_01_87 missing tile: blends_grassoverlays_01_16 missing tile: blends_street_01_86 missing tile: blends_grassoverlays_01_16

pzmap_error_0

pzmap_error_1

cff29546 commented 1 year ago

Have you got any error while unpacking textures? You can run scripts/unpack_texture.bat alone on a cmd console to check errors. You should get 17069 textures for the 41.78 vanilla game.

cff29546 commented 1 year ago

Also, how much free memory do you have on your computer? A typical 16 render threads setting will use about 16GB of memory. Tiles may not be able to load when you run out of memory. You can follow the instructions on the readme file to use fewer render threads, and that will reduce the memory needed.

verdra commented 1 year ago

No missing textures or errors. i got that many jobs as well, which i assume corresponds to textures.

i have 64 GB of ram so thats not a problem

cff29546 commented 1 year ago

I just fixed a bug that may cause the missing tile error. Can you try the latest code and see if the problem still exists?

verdra commented 1 year ago

ok running base over again and its looking good so far, let you know how it goes. had to increase my ulimit to get it to run.

verdra commented 1 year ago

also i am running on linux. i converted some of the batch scripts to python.

would you be open to PRs to convert the project to pure python to be more portable?

cff29546 commented 1 year ago

also i am running on linux. i converted some of the batch scripts to python.

would you be open to PRs to convert the project to pure python to be more portable?

I'm OK with PRs. The batch scripts are barely passing parameters to main entries. It is possible to add similar bash scripts for Linux support. Also, be aware that some libraries behave differently between Linux and Windows. For example, the shared_memory in Windows does not need unlink() to fully release as the OS does the clean-up when the last process closes the handle, but not calling unlink() accordingly on Linux will lead to a memory leak.

verdra commented 1 year ago

looks like it worked. thanks for the help.