You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
fix: sync eurac_pv_farm_detection udp description with the markdown docs #57 tries to fix this by duplicating the decsription from algorithm_catalog/eurac_pv_farm_detection/eurac_pv_farm_detection_description.md to openeo_udp/eurac_pv_farm_detection/README.md . Note that that readme references an image pv_ml_output.png which is not duplicated, so the latter readme is kind of broken. The image could also be duplicated of course, but duplication like this is of course not ideal and will lead to long term maintainability problems.
"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).
The text was updated successfully, but these errors were encountered:
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).
The text was updated successfully, but these errors were encountered: