Skip to content

Fix: UDPs not working#593

Open
firzaariany wants to merge 17 commits into
ESA-APEx:fixes_devseed_udpsfrom
developmentseed:fixes_devseed_udps
Open

Fix: UDPs not working#593
firzaariany wants to merge 17 commits into
ESA-APEx:fixes_devseed_udpsfrom
developmentseed:fixes_devseed_udps

Conversation

@firzaariany

@firzaariany firzaariany commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Related to #580

I needed to fix some UDPs to test in the browser. They all work well and standardize now. I confirmed that they work as intended and not bounded to a specific/hardcoded spatial and temporal bounds.

Even though I'm quite sure that re-target the branch to ESA-APEx:main would include all the changes from ESA-APEx:fixes_devseed_udps (2 commits) and developmentseed:fixes_devseed_udps (17 commits), I'm still hesitant to do it. I set developmentseed:fixes_devseed_udps to track the upstream branch with the same name. Need input on this

Issues:

  • Wildfire Visualization: Removes mask_polygon process with hard-coded bounds
  • Atmospheric Resistant Vegetation Index: Resolves output format definition
  • Enhanced Vegetation Index: Removes mask_polygon process with hard-coded bounds
  • Kernel NDVI: Add file format and cloud filter
  • LAI: Removes mask_polygon process with hard-coded bounds
  • PSRI: Resolves empty output file format and cloud filter

@firzaariany firzaariany changed the title Fixes devseed udps Fix: UDPs not working Jul 10, 2026
@algorithm-services-catalogue

Copy link
Copy Markdown

🔍 Catalogue's Preview Site Deployed

Your changes have been deployed to the preview site:

🔗 Preview URL: https://esa-apex.github.io/apex-algorithms-catalogue-web/pr-preview/pr-593/

This preview will be updated automatically when you push new changes to your PR.

@firzaariany
firzaariany marked this pull request as ready for review July 15, 2026 09:58
@firzaariany

firzaariany commented Jul 15, 2026

Copy link
Copy Markdown
Contributor Author

@JanssenBrm hi, sorry I had to fix some UDPs in a forked branch as they were not working cleanly. The problem wasn't due to removing the usage of the default parameters as you did in #580, but rather due to some processes were not integrated in the UDPs (e.g. save_result). What would be a cleaner way to merge this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant