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

Sentinel2 L1C: SAA SZA VAA VZA bands not found #177

Closed
EmileSonneveld opened this issue Jun 22, 2023 · 6 comments · Fixed by #185, #192, Open-EO/openeo-geotrellis-kubernetes#14 or #210
Closed
Assignees

Comments

@EmileSonneveld
Copy link
Contributor

EmileSonneveld commented Jun 22, 2023

Probably here, changing finder.creodias.eu to zipper.creodias.eu could solve the problem.
Also filtering out the prefixes could be nicer.

path.replace("/vsicurl/", "").replace("/vsis3/eodata", "https://finder.creodias.eu/files")

Example error from kibana:

java.net.MalformedURLException: no protocol: /eodata/Sentinel-2/MSI/L1C/2023/05/30/S2B_MSIL1C_20230530T104629_N0509_R051_T31UFS_20230530T125255.SAFE/GRANULE/L1C_T31UFS_A032538_20230530T104623/MTD_TL.xml
	at java.base/java.net.URL.<init>(URL.java:645)
	at java.base/java.net.URL.<init>(URL.java:541)
	at java.base/java.net.URL.<init>(URL.java:488)
	at org.openeo.geotrellis.layers.FileLayerProvider.rasterSource$1(FileLayerProvider.scala:947)
	at org.openeo.geotrellis.layers.FileLayerProvider.$anonfun$deriveRasterSources$21(FileLayerProvider.scala:1011)

The error is about the prefix, but even without the good prefix, the url gives a 404: https://finder.creodias.eu/files/Sentinel-2/MSI/L1C/2023/05/30/S2B_MSIL1C_20230530T104629_N0509_R051_T31UFS_20230530T125255.SAFE/GRANULE/L1C_T31UFS_A032538_20230530T104623/MTD_TL.xml

@EmileSonneveld EmileSonneveld self-assigned this Jun 22, 2023
EmileSonneveld added a commit that referenced this issue Jun 22, 2023
EmileSonneveld added a commit that referenced this issue Jul 3, 2023
…al conflicts when using S3. Allow to use S3 acces for angles layers. #177
@EmileSonneveld EmileSonneveld linked a pull request Jul 5, 2023 that will close this issue
EmileSonneveld added a commit that referenced this issue Jul 5, 2023
@EmileSonneveld EmileSonneveld linked a pull request Jul 14, 2023 that will close this issue
@jdries
Copy link
Contributor

jdries commented Aug 1, 2023

also check state of these bands SENTINEL2_L2A

EmileSonneveld added a commit that referenced this issue Aug 7, 2023
@EmileSonneveld EmileSonneveld linked a pull request Aug 7, 2023 that will close this issue
EmileSonneveld added a commit to Open-EO/openeo-geotrellis-kubernetes that referenced this issue Aug 7, 2023
EmileSonneveld added a commit to Open-EO/openeo-geotrellis-kubernetes that referenced this issue Aug 7, 2023
EmileSonneveld added a commit to openEOPlatform/documentation that referenced this issue Aug 7, 2023
EmileSonneveld added a commit to openEOPlatform/documentation that referenced this issue Aug 7, 2023
EmileSonneveld added a commit to openEOPlatform/documentation that referenced this issue Aug 7, 2023
EmileSonneveld added a commit that referenced this issue Aug 18, 2023
…l raster sources under BandCompositeRasterSource have the same row/col. #177
@jdries jdries reopened this Sep 12, 2023
@EmileSonneveld
Copy link
Contributor Author

This works now on CDSE and tests are added as to test_collections.py https://git.vito.be/projects/TPT/repos/os_creodias_openeo_k8s/browse/kube_resources/applications/integrationtests/tests/test_collections.py?at=797c76d2003343dbd42427fa8591c2777a1219ab#120-121

Also tested with example from Pratichhya: pratichhya_angle_bands.json.txt

However, on openeo.vito.be this does not work atm.

@EmileSonneveld
Copy link
Contributor Author

On openeo.vito.be those bands work: "relativeAzimuthAngles", "sunZenithAngles", "viewZenithAngles"
The sunZenithAngles has the same color distribution on both platforms:
image

@EmileSonneveld
Copy link
Contributor Author

This seem to work for simple graphs using angle bands, but when running this larger example, there is still an extent error:
https://github.com/jdries/worldwater-toolbox/blob/vito/tests/test_openeo_integration.py#L54

@EmileSonneveld
Copy link
Contributor Author

The graph from test_multiple_months() ran successfully locally on branch band_with_different_resolution:
image

@EmileSonneveld EmileSonneveld linked a pull request Sep 29, 2023 that will close this issue
@EmileSonneveld
Copy link
Contributor Author

Also tested on https://openeo.dev.warsaw.openeo.dataspace.copernicus.eu/ j-7113dc80887c47f08b04fc096954fdc1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment