Skip to content

Conversation

@JanssenBrm
Copy link
Contributor

Added an openEO e2e example where we explain the following steps:

  • Creating a workflow using openEO
  • Setting up and executing an experiment using the workflow
  • Publishing the experiment to EarthCODE
  • Reproducing the published experiment

@github-actions
Copy link

github-actions bot commented Aug 20, 2025

PR Preview Action v1.6.2
Preview removed because the pull request was closed.
2025-08-27 13:27 UTC

@u3ks u3ks requested a review from edobrowolska August 25, 2025 11:02
Copy link
Contributor

@edobrowolska edobrowolska left a comment

Choose a reason for hiding this comment

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

Hi @JanssenBrm Thank you for your work on these examples tutorials! I think they look great! I just left few comments/updates before merging, so please have a look. Also I think this is very nice set of notebooks to put into Getting started notebook and enrich the documentation. :) The publication part can also be referenced in the tutorials as a part of platform-specific publication tools. Thanks again!

Copy link
Contributor Author

@JanssenBrm JanssenBrm left a comment

Choose a reason for hiding this comment

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

Thank you @edobrowolska for this thorough review. Your feedback is greatly appreciated and very valuable. Your comments are absolutely valid and should now be addressed with the latest commit.

@u3ks
Copy link
Contributor

u3ks commented Aug 27, 2025

Just a few things from me:

  1. Can you merge main again to avoid a conflict with the below changes
  2. Can you add a paragraph to the root dir index.md file with something like: "Looking for how to use the OpenEO platform? Check out our OpenEO examples and link to openeo/index.md
  3. Add a short line and description of you publishing tool in the osc/index.md in the platform tools section, and link to the page with the publishing notebook.

Thanks for the detailed examples

@JanssenBrm
Copy link
Contributor Author

Thank you, @edobrowolska, for the additional feedback. I have added the necessary links in the other pages. Let me know if this PR can be merged into the main branch or if there are other things we should adapt before doing so.

Copy link
Contributor

@edobrowolska edobrowolska left a comment

Choose a reason for hiding this comment

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

@JanssenBrm from my side it's fine to merge. Thank you for addressing my comments it looks great! :)

@u3ks u3ks merged commit 4a4e4cb into main Aug 27, 2025
1 check passed
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.

4 participants