From 32397399f282cbc895b8692c87bec51090d7a9e2 Mon Sep 17 00:00:00 2001 From: James Ousby Date: Mon, 3 Aug 2026 21:16:42 +1000 Subject: [PATCH] Fix --build-overviews: add missing space in gdaladdo arguments Adjacent string literals are concatenated at compile time, so the missing trailing space after YCBCR made gdaladdo receive the orthophoto path as the value of --config PHOTOMETRIC_OVERVIEW: ... --config PHOTOMETRIC_OVERVIEW YCBCR/path/to/odm_orthophoto.tif 2 4 8 16 gdaladdo then treats "2" as its input raster and exits non-zero, which system.run turns into a SubprocessException, aborting the run at the odm_orthophoto stage. Introduced in #1935; affects 3.6.0 and 3.6.1. Fixes #2058 Co-Authored-By: Claude Opus 5 (1M context) --- opendm/orthophoto.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/opendm/orthophoto.py b/opendm/orthophoto.py index 27f42b81a..74d44c81b 100644 --- a/opendm/orthophoto.py +++ b/opendm/orthophoto.py @@ -38,7 +38,7 @@ def build_overviews(orthophoto_file): '--config BIGTIFF_OVERVIEW IF_SAFER ' '--config COMPRESS_OVERVIEW JPEG ' '--config INTERLEAVE_OVERVIEW PIXEL ' - '--config PHOTOMETRIC_OVERVIEW YCBCR' + '--config PHOTOMETRIC_OVERVIEW YCBCR ' '{orthophoto} 2 4 8 16'.format(**kwargs)) def generate_png(orthophoto_file, output_file=None, outsize=None):