Skip to content

entry rayview#508

Open
HansVRP wants to merge 8 commits into
mainfrom
hv_solarray
Open

entry rayview#508
HansVRP wants to merge 8 commits into
mainfrom
hv_solarray

Conversation

@HansVRP

@HansVRP HansVRP commented May 21, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@algorithm-services-catalogue

algorithm-services-catalogue Bot commented May 21, 2026

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-508/

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

@HansVRP HansVRP assigned HansVRP and unassigned JanssenBrm May 26, 2026
@HansVRP

HansVRP commented Jun 24, 2026

Copy link
Copy Markdown
Contributor Author

@JanssenBrm PR is ready to be merged; few failing unit test which are unrelated

@JanssenBrm

Copy link
Copy Markdown
Contributor

Thanks for the record @HansVRP! Some minor feedback:

  • The description on the record is very minimal. I can see algorithm_catalog/ucl/solar_pv_detection/openeo_udp/solar_pv_detection_description.md but the algorithm catalogue is reading the description from the UDP itself. This ensures that the description is consistent across tooling using the UDPs.
  • The bbox parameter is defined as an object. This is causing strange behaviour in the openEO web editor and will also cause errors in other tools. Can this be updated to use the openEO bounding box parameter?
  • Similar as above for the end date, maybe it is better to use the date parameter to ensure that a nice datepicker is shown?

Out of my interest: there is an end_date parameter, so I would also expect a start date? Unless I've missed it, the documentation does not clearly state what will happen. Does it process the full archive until the end_date?

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.

2 participants