-
Notifications
You must be signed in to change notification settings - Fork 1
Tiler #24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
Seems to be working now - # %%
# skyview factor for SOLWEIG - tiled
skyviewfactor_algorithm.generate_svf(
dsm_path=input_path_str + "/DSM.tif",
bbox=total_extents,
out_dir=output_folder_path_str + "/svf_tiled",
dem_path=input_path_str + "/DEM.tif",
cdsm_path=output_folder_path_str + "/CDSM.tif",
trans_veg_perc=3,
use_tiled_loading=True,
tile_size=200,
)
# %%
# Tiled
SRC = solweig_runner_core.SolweigRunCore(
"demos/data/athens/configsolweig_tiled.ini",
"demos/data/athens/parametersforsolweig.json",
use_tiled_loading=True,
tile_size=200,
)
SRC.run()PS - 200 used for testing but 1000 or 2000 probably better range to chunk |
|
PS - the motivation with this change is to handle large files which would sometimes cause machine to run out of RAM. It also makes it easier for the downstream implementation in the rust version to use the GPU, which would otherwise be overwhelmed the GPU buffers for large files. In combination with the |
|
The shadow matrices (SVF) for anisotropic sky were still prohibitively large so these are now converted to |
|
Strange. I Though I left a comment here but it seem to have disappeared. Your buffer parameter specifying the overlap between the tiles, is it 150 one way or both ways (i.e. merging at 75 meters or 150 meters)? How do you merge (just sharp transition at buffer edge of do you blend)? |
Experimental tiled version for processing large extents