Guido Riembauer

> The problem with `db.execute input="-"` is the statically defined character buffer size if the SQL string is read from stdin `input="-"`. > > https://github.com/OSGeo/grass/blob/ea8f3eae3299d70be3e46f2d1beeb9a72635e525/db/db.execute/main.c#L189-L191 > > https://github.com/OSGeo/grass/blob/ea8f3eae3299d70be3e46f2d1beeb9a72635e525/include/grass/dbmi.h#L142 > >...

> Probably a set of SQL commands should be wrapped into a [TRANSACTION](https://www.sqlite.org/lang_transaction.html)? > > ``` > sql = ["BEGIN TRANSACTION"] > ... > sql.append("END TRANSACTION") > ``` > >...

Hi @ninsbl, we finally found time to look into your great addons! I haven't managed to run `i.sentinel1.pyrosargeocode` yet, maybe you could advise: I am trying `i.sentinel1.pyrosargeocode -fndm input=/data/s1_input/S1A_IW_GRDH_1SDV_20230807T181458_20230807T181523_049773_05FC30_73BF.SAFE.zip output_directory=/data/s1_output...

Update: It seems that this specific error is caused by the value of `externalDEMFile` in the `kwargs` dict: ``` (Pdb) geocode_kwargs["externalDEMFile"] PosixPath('/data/dem/ireland_dem_native_int_utm29n.tif') ``` which comes from the `Path(...` calls in...

Update: Moving the discussion to https://github.com/NVE/actinia_modules_nve/issues/34

After some tests I would recommend to go ahead with @ninsbl really great addon `i.sentinel1.pyrosargeocode` as the new standard for S1-Preprocessing in the actinia image. Snappy does not work with...

Hi @wenzeslaus, we finally implemented the requested changes (some code restructuring & commenting, an example in the .html, added a test). Could you re-review? Thanks!