DTTrierFlood/GeoData
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

conductor(plan): Mark task 'Refactor _run' as complete

Browse Source
This commit is contained in:
Matthias Puchstein
2026-02-03 23:09:44 +01:00
parent 358e093e79
commit 3d05459a2d
1 changed files with 17 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
conductor/tracks/fix_memory_subprocess_20260203/plan.md Normal file
View File

@@ -0,0 +1,17 @@
# Implementation Plan - Fix Critical Memory & Subprocess Issues
## Phase 1: Fix Subprocess Handling in buildings.py
- [x] Task: Refactor `_run` in `geodata_pipeline/buildings.py` (358e093)
- [ ] Update `_run` to remove unsafe splitting of arguments.
- [ ] Ensure all callers pass lists of strings correctly.
## Phase 2: Fix Memory Leak in buildings.py
- [ ] Task: Implement windowed reading for ground snapping
- [ ] Calculate `srcwin` (xoff, yoff, xsize, ysize) based on tile bounds and VRT geotransform.
- [ ] Update `ReadAsArray` to use the calculated window.
- [ ] Handle edge cases (partial overlap).
## Phase 3: Optimize Heightmap Export (Optional)
- [ ] Task: Use in-memory warp in `geodata_pipeline/heightmaps.py`
- [ ] Replace temporary file output in `gdal.Warp` with `MEM` driver.
- [ ] Pass the in-memory dataset to the next processing step.