This came up while reviewing a couple of description fixup PRs by @rahmandawibowo-vito :
It looks we're currently quite inconsistent with how algorithm and/or descriptions are handled:
- "eurac_pv_farm_detection":
- "sentinel1_stats": same kind of problem as "eurac_pv_farm_detection"
- "max_ndvi_composite" uses different solution: the UDP generate.py script explicitly links to the description markdown file from the algorithm description algorithm_catalog/max_ndvi_composite_files/max_ndvi_composite_description.md. Which eliminates the duplication problem, but instead causes a weird coupling issue: the UDP depends on a resource from the algorithm folder, while the algorithm itself actually doesn't directly use this resource.
tl;dr I think there is kind of confusion about whether what is described (algorithm vs UDP) and where it should be described (algorithm or UDP related folders).
This came up while reviewing a couple of description fixup PRs by @rahmandawibowo-vito :
It looks we're currently quite inconsistent with how algorithm and/or descriptions are handled:
tl;dr I think there is kind of confusion about whether what is described (algorithm vs UDP) and where it should be described (algorithm or UDP related folders).