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

added wizard section to openEO web Editor documentation #233

Merged
merged 9 commits into from
Dec 22, 2023

Conversation

Pratichhya
Copy link
Collaborator

Updated version of #163

Adopted the suggested changes

#163 (comment)

#163 (comment)

#163 (comment)

@Pratichhya Pratichhya mentioned this pull request Jul 28, 2023
@github-actions
Copy link

github-actions bot commented Jul 28, 2023

PR Preview Action v1.4.4
🚀 Deployed preview to https://eu-cdse.github.io/documentation/pr-preview/pr-233/
on branch gh-pages at 2023-12-08 14:08 UTC

@Pratichhya Pratichhya linked an issue Aug 8, 2023 that may be closed by this pull request
@soxofaan
Copy link
Collaborator

That GIF animation is quite large, both in byte size and in pixels (1800px wide). This is quite heavy to load and makes the actual text hard to read because the image will be shrunk in the docs (e.g. to fit in column width).

I'd consider making these kind of screen captures with a non-fullscreen browser window, like the smallest window with that preserves the normal layout of the main elements of the page or feature you want to show.

@Pratichhya
Copy link
Collaborator Author

That GIF animation is quite large, both in byte size and in pixels (1800px wide). This is quite heavy to load and makes the actual text hard to read because the image will be shrunk in the docs (e.g. to fit in column width).

I'd consider making these kind of screen captures with a non-fullscreen browser window, like the smallest window with that preserves the normal layout of the main elements of the page or feature you want to show.

Ok sure. I agree with you. Also modification are required to these GIFs since they reflect the old editor version. Thus created a separate issue for it too #261.

@Pratichhya Pratichhya linked an issue Sep 1, 2023 that may be closed by this pull request
@Pratichhya Pratichhya changed the title added wizard section to openEo web Editor documentation added wizard section to openEO web Editor documentation Sep 3, 2023
@Pratichhya
Copy link
Collaborator Author

That GIF animation is quite large, both in byte size and in pixels (1800px wide). This is quite heavy to load and makes the actual text hard to read because the image will be shrunk in the docs (e.g. to fit in column width).

I'd consider making these kind of screen captures with a non-fullscreen browser window, like the smallest window with that preserves the normal layout of the main elements of the page or feature you want to show.

I have updated all the gifs and tried to maintaining the size of the file to less than 1MB. Nevertheless, I am not sure if the quality of available GIFs is good to be integrated or not.

@Pratichhya
Copy link
Collaborator Author

To maintain a clean and clear view, I changed all the GIFs to images instead. Please let me know if this view is better or should change to GIF itself instead.

@soxofaan
Copy link
Collaborator

soxofaan commented Oct 3, 2023

Yeah, it's probably better to stick to static images where possible, as animated gifs are harder to review/maintain.

Small remark tough: I notice that a lot of these new images are in the root of the _images folder and have very short generic names like "save.png", "bbox.png", "bands.png", while they are specific to the webeditor. I think it is more future-proof to give a more specific name, or put them in a subfolder.

@Pratichhya
Copy link
Collaborator Author

Yeah, it's probably better to stick to static images where possible, as animated gifs are harder to review/maintain.

Small remark tough: I notice that a lot of these new images are in the root of the _images folder and have very short generic names like "save.png", "bbox.png", "bands.png", while they are specific to the webeditor. I think it is more future-proof to give a more specific name, or put them in a subfolder.

Yes, I agree. Thank you for suggestion, it makes sense. However, in this PR, I rearranged only the images of the web editor section , but I will create a new PR to rearrange images.

@Pratichhya Pratichhya mentioned this pull request Oct 27, 2023
@@ -54,7 +54,7 @@ Upon initial access to the provided link, users are presented with the following

As previously mentioned, it is necessary to log in to interact with the server. A new window will appear when attempting to log in, as demonstrated below. While other options are sometimes available, the recommended authentication choice is the "Copernicus Data Space Ecosystem". For further information regarding various authentication methods or to seek assistance, you can always click on the "help" option at the top or contact us.

<centre><img src="_images/login.PNG" width="80%" height="60%"></centre>
<centre><img src="_images/webeditor/editor_login.PNG" width="80%" height="60%"></centre>
Copy link
Collaborator

Choose a reason for hiding this comment

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

note that this image is pretty outdated, it still contains "EGI" and "internal" options, which are dropped now

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

updated the image

@@ -63,48 +63,96 @@ Based on their applications user can build their model by simple drag and drop m

1. Load Collection

In order to load the required collection make sure it is available by searching in the sidebar. Once you find your collection you can simply drag and drop it for carrying out further actions. In the following clip, you can observe the sequential actions are taken to accomplish the following tasks: choosing a collection of interest, defining the spatial and temporal boundaries, and filtering the necessary bands for subsequent processing. Specifically, for calculating the NDVI, the Red band (B04) and Near-Infrared (NIR) band (B08) have been selected.
a. To start a task within the openEO web editor, your first step involves ensuring you have the necessary dataset for the analysis. Thus, you must ensure the required dataset exists within the openEO database. You can do this by exploring/searching the list of available datasets from the sidebar of the interface.
Copy link
Collaborator

Choose a reason for hiding this comment

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

use "collection" iso "dataset". Also "ensure" sounds a bit weird here

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

corrected

In order to load the required collection make sure it is available by searching in the sidebar. Once you find your collection you can simply drag and drop it for carrying out further actions. In the following clip, you can observe the sequential actions are taken to accomplish the following tasks: choosing a collection of interest, defining the spatial and temporal boundaries, and filtering the necessary bands for subsequent processing. Specifically, for calculating the NDVI, the Red band (B04) and Near-Infrared (NIR) band (B08) have been selected.
a. To start a task within the openEO web editor, your first step involves ensuring you have the necessary dataset for the analysis. Thus, you must ensure the required dataset exists within the openEO database. You can do this by exploring/searching the list of available datasets from the sidebar of the interface.

b. In our example, we want to calculate the Normalized Difference Vegetation Index (NDVI), so the Sentinel 2 L2A collection will be used. Once you find this collection, the next step is to drag and drop it into the Process Editor for further operations.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't understand the difference between step a and b here, both are about finding the desired collection.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

combined and modified all the steps

![](_images/webeditor/drag_collection.png)


Now, once you load collection, there are several parameters you need to specify, including the area of interest, the temporal extent, and the selection of bands. When you click on the collection, a window pops up, where you can define the necessary parameters for subsequent processing.
Copy link
Collaborator

Choose a reason for hiding this comment

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

When you click on the collection, a window pops up,

I think this should be clicking on any load_collection parameter, or clicking the "edit" icon in top right corner

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

updated the sentence


d. Another parameter is Temporal extent, that imits the data to load from the collection to the specified temporal interval.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
d. Another parameter is Temporal extent, that imits the data to load from the collection to the specified temporal interval.
d. Another parameter is temporal extent to limit the data to load to the specified time window.


2. Apply Processes

Eventually, the next step involves implementing essential processes, ranging from straightforward operations like adding bands to more complex tasks such as importing or defining user-defined functions (UDFs). In the following clip, a reduce_dimension() process is employed to eliminate the temporal dimension by selecting the maximum value. The same process is utilised to reduce the band dimension after executing a series of addition, subtraction, and division operations necessary for the NDVI calculation.
Eventually, the next step involves implementing essential processes, ranging from straightforward operations like adding bands to more complex tasks such as importing or defining user-defined functions (UDFs).
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Eventually, the next step involves implementing essential processes, ranging from straightforward operations like adding bands to more complex tasks such as importing or defining user-defined functions (UDFs).
The next step involves applying essential openEO processes, ranging from straightforward mathematical operations to more complex tasks such spatial or temporal aggregations.

Eventually, the next step involves implementing essential processes, ranging from straightforward operations like adding bands to more complex tasks such as importing or defining user-defined functions (UDFs). In the following clip, a reduce_dimension() process is employed to eliminate the temporal dimension by selecting the maximum value. The same process is utilised to reduce the band dimension after executing a series of addition, subtraction, and division operations necessary for the NDVI calculation.
Eventually, the next step involves implementing essential processes, ranging from straightforward operations like adding bands to more complex tasks such as importing or defining user-defined functions (UDFs).

In this task, we're using a specific process called `reduce_dimension` as a key part of the operation. This process helps us simplify our collection: first, we use it to reduce temporal dimension and second, to deal with the various bands.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
In this task, we're using a specific process called `reduce_dimension` as a key part of the operation. This process helps us simplify our collection: first, we use it to reduce temporal dimension and second, to deal with the various bands.
In this task, we'll using a specific openEO process called `reduce_dimension` two times to simplify our data cube. First to reduce the temporal dimension, and second to deal with the various bands.


![](_images/webeditor/reduce_dimension.png)

In the initial process, the `reduce_dimension` algorithm eliminates the temporal dimension by selecting the maximum value using the max function.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Note that, scientifically speaking, it is pretty weird to first do a temporal reduction of your raw sensor values and do NDVI calculation afterwards because you will be calculating the NDVI from red and nir values that might be unrelated. I think you first should do NDVI calculation and then taking the temporal max.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

updated the method and noted


![](_images/webeditor/save_process.png)

Since our application involves simple NDVI calculation, we want to save it as a GeoTiff.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Since our application involves simple NDVI calculation, we want to save it as a GeoTiff.
Since our workflow eliminated the temporal dimension, we can keep things simple and just save the result as a GeoTiff.


Now, once you load collection, there are several parameters you need to specify, including the area of interest, the temporal extent, and the selection of bands. When you click on the collection, a window pops up, where you can define the necessary parameters for subsequent processing.

c. To specify the area of interest, you can choose between generating a bounding box or importing your spatial extent by dragging and dropping GeoJSON or KML files onto the map view. Here, we've provided an example involving a bounding box. There is also an option to "No filter", that will include all the data in the datacube.
Copy link
Collaborator

Choose a reason for hiding this comment

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

No bounding box filter is generally a bad idea for novice users (the ones that use a wizard). Instead we should point out that the user should start with a sufficiently small boundingbox with respect to the resolution of the data.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

updated the sentence


d. Another parameter is Temporal extent, that imits the data to load from the collection to the specified temporal interval.
Copy link
Collaborator

Choose a reason for hiding this comment

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

here as well, same as bbox: please encourage the user to start small, like a 10 or 15 day interval.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

updated the sentence

@Pratichhya
Copy link
Collaborator Author

Assuming that the updated changes are good to go, please allow me to merge this PR to prevent conflict when rearranging the file for introductory page visualization modification.

Please, additional suggestions; please kindly reopen the connected issue.

@Pratichhya Pratichhya merged commit ecdec11 into publish Dec 22, 2023
1 check passed
@Pratichhya Pratichhya deleted the openEO_webeditor_v2 branch January 15, 2024 09:17
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.

old gif in web-editor Include wizard feature in openEO documentation
3 participants