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

Sentinel and Landsat processing in tutorials #261

Open
juleshaas opened this issue Oct 7, 2021 · 4 comments
Open

Sentinel and Landsat processing in tutorials #261

juleshaas opened this issue Oct 7, 2021 · 4 comments
Assignees
Labels
documentation related to all kinds of docs (API docs, website, ...) prio

Comments

@juleshaas
Copy link
Member

Sentinel processing does not work due to GoogleCloudAPIError. This happens in tutorial_sentinel2_ndvi.md and in tutorial_process_chain.md.

The Landsat processing in tutorial_landsat_ndvi results in KeyError. GoogleCloudAPIError will probabaly also be an issue here.

The errors are documented in: #214 Docs review.

The Landsat/Sentinel examples are also used in other documentation (e.g. actinia_core/docs/docs/introduction.md, actinia website).

@mmacata
Copy link
Member

mmacata commented Nov 11, 2021

Can be fixed with #242

@griembauer
Copy link
Contributor

griembauer commented Jul 28, 2022

In the tutorials, the Sentinel-2 NDVI is calculated in two ways using
a) a process chain with an importer and a r.mapcalc command: https://github.com/mundialis/actinia_core/blob/main/docs/docs/tutorial_process_chain.md
b) the ndvi endpoint of the actinia_satellite_plugin: https://github.com/mundialis/actinia_core/blob/main/docs/docs/tutorial_sentinel2_ndvi.md

a) is now in possible due to changes in #295. However, a bug in i.sentinel.import does currently not allow import into lat/lon locations, which is used in the tutorial (and the demouser is not allowed to create a new location). This is fixed here: OSGeo/grass-addons#769. This will only become available to actinia after it has been merged in grass_addons and a new actinia release has been created

b) is not yet possible as the actinia_satellite_plugin has not been adapted yet

Landsat processing is not fixed yet so it is ignored at the moment.

@griembauer
Copy link
Contributor

actinia.mundialis.de is now on actinia-core version 4.1.0. This means that S-2 importing works on locations with projection units meters. In #363, the tutorials are adapted such that the process-chain example uses the nc_spm_08 location, while the tutorials using the ndvi endpoint of the actinia-satellite-plugin are temporarily removed.

Still TODO: adapt actinia-satellite-plugin and update tutorials accordingly.

@metzm
Copy link
Contributor

metzm commented Aug 3, 2022

As a reminder: the jupyter notebook versions of the tutorials need to be kept in sync with the md versions.

metzm added a commit to actinia-org/actinia-jupyter that referenced this issue Aug 3, 2022
Sentinel-2 NDVI and Landsat NDVI examples are currently not working because of actinia-org/actinia-core#261
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation related to all kinds of docs (API docs, website, ...) prio
Projects
None yet
Development

No branches or pull requests

5 participants