Change default of tpi inner radious to 0

awi-response · Nov 21, 2024 · 8aaf812 · 8aaf812
1 parent f6bcda1
commit 8aaf812
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/config.toml.example b/config.toml.example
@@ -16,8 +16,8 @@ arcticdem-dir = "data/download/arcticdem"
 cache-dir = "data/download"
 
 [darts.preprocess]
-tpi-outer-radius = 30
-tpi-inner-radius = 25
+tpi-outer-radius = 32 # px. Original implementation was 100m (100 / 3.125 = 32)
+tpi-inner-radius = 0
 
 [darts.segmentation]
 patch-size = 1024

diff --git a/darts-preprocessing/src/darts_preprocessing/engineering/arcticdem.py b/darts-preprocessing/src/darts_preprocessing/engineering/arcticdem.py
@@ -10,14 +10,14 @@
 
 
 def calculate_topographic_position_index(
-    arcticdem_ds: xr.Dataset, outer_radius: int = 30, inner_radius: int = 25
+    arcticdem_ds: xr.Dataset, outer_radius: int = 30, inner_radius: int = 0
 ) -> xr.Dataset:
     """Calculate the Topographic Position Index (TPI) from an ArcticDEM Dataset.
 
     Args:
         arcticdem_ds (xr.Dataset): The ArcticDEM Dataset containing the 'dem' variable.
         outer_radius (int, optional): The outer radius of the annulus kernel in number of cells. Defaults to 30.
-        inner_radius (int, optional): The inner radius of the annulus kernel in number of cells. Defaults to 25.
+        inner_radius (int, optional): The inner radius of the annulus kernel in number of cells. Defaults to 0.
 
     Returns:
         xr.Dataset: The input Dataset with the calculated TPI added as a new variable 'tpi'.