Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Simple example of Multipleresults #65

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Simple example of Multipleresults #65

wants to merge 4 commits into from

Conversation

Pratichhya
Copy link
Contributor

@Pratichhya Pratichhya commented Feb 3, 2025

#51

Similar task is also in progress with: #64

@Pratichhya Pratichhya requested a review from HansVRP February 3, 2025 13:37
@Pratichhya Pratichhya linked an issue Feb 3, 2025 that may be closed by this pull request
Copy link
Contributor

@HansVRP HansVRP left a comment

Choose a reason for hiding this comment

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

I would not keep both examples, and simply use the one where mutiple results are saved.

It would be nice if you could include an image of the process graph prior to execution and provide some details on how this graph now contains multiple saving nodes which are all processed and saved at once

@Pratichhya
Copy link
Contributor Author

I would not keep both examples, and simply use the one where mutiple results are saved.

Included the feature in same example.

It would be nice if you could include an image of the process graph prior to execution and provide some details on how this graph now contains multiple saving nodes which are all processed and saved at once

Updated

@@ -27,7 +27,7 @@ The `Demonstrates` column summarizes the key openEO functionality used in each c
| [WorldCereal](./WorldCereal/) | `Jupyter` | WorldCereal data extraction sample. | openEO processes `merge_cubes`, loading **WorldCereal** data |
| [RVI](./RVI/) | `Jupyter` | Calculate Radar Vegetation Index | openEO processes `sar_backscatter`, `spectral_nidices.compute_indices`; **plotting** mean result and timeseries; **Awesome Spectral Indices** |
| [OilSpill](./OilSpill/) | `Jupyter` | Oil Spill mapping with Sentinel-1 layer. | openEO processes `sar_backscatter`, `apply`, `apply_kernel`, `rename_labels`, `merge_cubes`; **plotting** binary image |
| [ForestFire](./ForestFire/) | `Jupyter` | Wildfire mapping using Sentinel-2 | openEO processes `apply_kernel`,`ndvi` `spectral_nidices.compute_indices`; **plotting** comparative visualisation; **Awesome Spectral Indices** |
| [ForestFire](./ForestFire/) | `Jupyter` | Wildfire mapping using Sentinel-2 | openEO processes `apply_kernel`,`ndvi`,`spectral_nidices.compute_indices`,`MultiResult`; **plotting** comparative visualisation; **Awesome Spectral Indices** |
Copy link
Contributor

Choose a reason for hiding this comment

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

I would reconsider this title, ideally it hints towards both the application as the openeo functionality used (saving multiple results)

"Forest Fire Analysis: Saving Multiple Results in One Job" or so?

Copy link
Contributor

@HansVRP HansVRP left a comment

Choose a reason for hiding this comment

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

In general the notebook is quite convoluted, please take a closer look from story telling perspective on how you can impove it. I would definetely make some choices and try to shorten it.

the introduction is not coherent, and when the process graph is shown, I do not believe we discuss the multiple save result nodes?

There is also some code replication going on with the load collection, it could help to wrap this in order to avoid repeating yourself

"Instead of executing on the backend immediately to get the ndvi_pre as an output, let us save it as an intermediate result with a specific name and format for the moment." I would rephrase towars something like: "since we are interested in this intermediant result, we attach a save result ....."

Personally I find

(i) NRT mode and
(ii) Post-event mode.

a bit confusing, do we need both?

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.

Create example on using multiple save results
2 participants