Skip to content

Add BAIS2 UDP entry for Development Seed#316

Open
zacdezgeo wants to merge 7 commits intoESA-APEx:mainfrom
developmentseed:alg/bais2
Open

Add BAIS2 UDP entry for Development Seed#316
zacdezgeo wants to merge 7 commits intoESA-APEx:mainfrom
developmentseed:alg/bais2

Conversation

@zacdezgeo
Copy link

Create provider record, add BAIS2 service record and UDP JSON, and include preview/thumbnail images from the notebook output.

@zacdezgeo zacdezgeo marked this pull request as draft January 21, 2026 11:40
@HansVRP HansVRP requested a review from JanssenBrm January 27, 2026 14:56
Create provider record, add BAIS2 service record and UDP JSON, and include preview/thumbnail images from the notebook output.
@zacdezgeo zacdezgeo marked this pull request as ready for review January 29, 2026 07:25
"href": "https://github.com/developmentseed/openeo-udp"
},
{
"rel": "about",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can also use the notebook as rel type. This would redirect the user to an APEx JupyterLite instance which will load in the notebook for the user to execute. However, it could be good to double check if the notebook is actually running properly.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just noting that i need to come back to this one.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated in recent commit pointing to: https://raw.githubusercontent.com/developmentseed/openeo-udp/refs/heads/main/notebooks/sentinel/sentinel-2/fire_and_disaster_monitoring/bais2_burned_area.ipynb. cc: @emmanuelmathot

"license": "CC-BY-4.0"
},
"linkTemplates": [],
"links": [
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there also a webapp link that we could add, similar to the other UDPs? This would allow the user to directly navigate to a UI for executing the service.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just noting that i need to come back to this one.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added link pointing to: "https://editor.openeo.org/?wizard=UDP&wizard%7Eprocess=bais2&wizard%7EprocessUrl=https://raw.githubusercontent.com/ESA-APEx/apex_algorithms/refs/heads/main/algorithm_catalog/developmentseed/bais2/openeo_udp/bais2.json&server=openeo.cloud

I followed the pattern for the other udps, but i don't think this will be available until we make a copy of this branch on this repository directly.

@JanssenBrm
Copy link
Contributor

Thank you @zacdezgeo for this initial PR and setting up the records. My apologies for the many comments, but it seems that the records are still requiring some information to pass the unit test.

I see that our documentation is missing a part where you can execute the unit tests locally, but I did this by following these steps:

  1. Installing the APEx QA tooling by running pip install qa/tools
  2. Executing the validation of the records through pytest qa/unittests/tests/test_records.py

I would expect the unit tests to be running from the PR automatically, but it seems that they are not getting started...

Unfortunately, there is also a known bug where the preview function of the catalogue is not working from remote forks. However, I suggest that once the records have been updated, we duplicate the branch to this repository, allowing you to visually inspect how the record would look like in the catalogue.

Add TiTiler openEO platform record and link BAIS2
@zacdezgeo
Copy link
Author

Thank you so much for the detailed review and for taking the time to leave all of these comments. I really appreciate it.

This is my first contribution to the apex_algorithms repository, and your feedback is extremely helpful for understanding the expected schema requirements (especially around provider metadata and link conventions). It’s been a great guide for getting the records into the shape needed to pass the unit tests and display correctly in the catalogue.

I’ll work through the requested updates and rerun the QA validation locally with pytest qa/unittests/tests/test_records.py before pushing the next commit.

Thanks again for the support in navigating the contribution process here, it’s very appreciated.

@zacdezgeo zacdezgeo marked this pull request as draft January 29, 2026 16:11
@zacdezgeo
Copy link
Author

I followed up on your comments regarding the record's links, but I still need to add the platform's logo (dark and light) based on the results from running: pytest qa/unittests/tests/test_records.py. I'll follow-up.

@zacdezgeo zacdezgeo marked this pull request as ready for review February 7, 2026 14:41
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