actinia-org / actinia-core

Actinia Core is an open source REST API for scalable, distributed, high performance processing of geographical data that uses mainly GRASS GIS for computational tasks (DOI: https://doi.org/10.5281/zenodo.5879231) | Tutorial: https://actinia-org.github.io/actinia-core/ | Docker: https://hub.docker.com/r/mundialis/actinia-core
https://actinia.mundialis.de/
GNU General Public License v3.0
78 stars 23 forks source link

Sentinel and Landsat processing in tutorials #261

Open juleshaas opened 2 years ago

juleshaas commented 2 years ago

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 commented 2 years ago

Can be fixed with https://github.com/mundialis/actinia_core/issues/242

griembauer commented 2 years ago

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 https://github.com/mundialis/actinia_core/pull/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: https://github.com/OSGeo/grass-addons/pull/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 commented 2 years ago

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 https://github.com/mundialis/actinia_core/pull/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 commented 2 years ago

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