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.

Sign up for GitHub

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

Jump to bottom

resample_spatial of SENTINEL3_SYN_L2_SYN #920

Closed
bossie opened this issue Oct 25, 2024 · 1 comment · Fixed by #923 or Open-EO/openeo-geotrellis-extensions#341
Closed

resample_spatial of SENTINEL3_SYN_L2_SYN #920

bossie opened this issue Oct 25, 2024 · 1 comment · Fixed by #923 or Open-EO/openeo-geotrellis-extensions#341
Assignees
@bossie

Comments

@bossie
Copy link
Collaborator

bossie commented Oct 25, 2024

https://forum.dataspace.copernicus.eu/t/spatial-resampling-causes-arraymemoryerror-and-catalogue-validation-error/1288/3
@bossie bossie self-assigned this Oct 25, 2024
bossie added a commit to Open-EO/openeo-geotrellis-extensions that referenced this issue Oct 29, 2024
@bossie
consider upper temporal bound exclusive with backwards compatibility
6995a6f 
Open-EO/openeo-geopyspark-driver#920
bossie added a commit that referenced this issue Oct 29, 2024
@bossie
fix tests
99137d5 
#920
bossie added a commit that referenced this issue Oct 29, 2024
@bossie
quick fix temporal resolution of layer metadata
8b36ace 
Depends on changes in openeo-geotrellis-extensions.

#920
@bossie bossie mentioned this issue Oct 29, 2024
Closed
@bossie bossie linked a pull request Oct 29, 2024 that will close this issue
enforce actual call to resample_spatial #922
Closed
bossie added a commit that referenced this issue Oct 29, 2024
@bossie
works without resample_spatial
2839e70 
#920
bossie added a commit that referenced this issue Oct 29, 2024
@bossie
actual resolution does not matter, will be resampled anyway
f7cc5f4 
#920
bossie added a commit that referenced this issue Oct 29, 2024
@bossie
update main
1c48f1b 
#920
bossie added a commit that referenced this issue Oct 29, 2024
@bossie
debug
7034455 
#920
bossie added a commit that referenced this issue Oct 29, 2024
@bossie
fix tests
f54e0f0 
#920
bossie added a commit that referenced this issue Oct 29, 2024
@bossie
quick fix temporal resolution of layer metadata
4f62c90 
Depends on changes in openeo-geotrellis-extensions.

#920
@bossie bossie mentioned this issue Oct 29, 2024
Merged
@bossie bossie linked a pull request Oct 29, 2024 that will close this issue
enforce actual call to resample_spatial #923
Merged
@bossie bossie mentioned this issue Oct 29, 2024
Merged
@bossie bossie linked a pull request Oct 29, 2024 that will close this issue
consider upper temporal bound exclusive with backwards compatibility Open-EO/openeo-geotrellis-extensions#341
Merged
bossie added a commit to Open-EO/openeo-geotrellis-extensions that referenced this issue Oct 29, 2024
@bossie
add test
2bda1a3 
Open-EO/openeo-geopyspark-driver#920
bossie added a commit to Open-EO/openeo-geotrellis-extensions that referenced this issue Oct 30, 2024
@bossie
fix upper temporal bound in layer metadata
57761e2 
Open-EO/openeo-geopyspark-driver#920
bossie added a commit to Open-EO/openeo-geotrellis-extensions that referenced this issue Oct 30, 2024
@bossie
DRY
d6d14e9 
Open-EO/openeo-geopyspark-driver#920
bossie added a commit to Open-EO/openeo-geotrellis-extensions that referenced this issue Oct 30, 2024
@bossie
DRY
fa7ef94 
Open-EO/openeo-geopyspark-driver#920
bossie added a commit that referenced this issue Oct 30, 2024
@bossie
restore Orfeo implementation
80b8c7f 
#920
bossie added a commit that referenced this issue Oct 30, 2024
@bossie
restore SENTINEL3_SLSTR_L2_LST resolution
1cd2185 
#920
bossie added a commit to Open-EO/openeo-geotrellis-extensions that referenced this issue Oct 30, 2024
@bossie
fix upper temporal bound in layer metadata
114c787 
* consider upper temporal bound exclusive with backwards compatibility

Open-EO/openeo-geopyspark-driver#920

* add test

Open-EO/openeo-geopyspark-driver#920

* fix upper temporal bound in layer metadata

Open-EO/openeo-geopyspark-driver#920

* DRY

Open-EO/openeo-geopyspark-driver#920
bossie added a commit that referenced this issue Oct 30, 2024
@bossie
cleanup
bb8871a 
#920
bossie added a commit that referenced this issue Oct 30, 2024
@bossie
DRY
c484df2 
#920
bossie added a commit that referenced this issue Oct 30, 2024
@bossie
adapt version and CHANGELOG #920
2edcf4d
bossie added a commit that referenced this issue Oct 30, 2024
@bossie
fix resample_spatial of Sentinel-3 data cube
20c731e 
* works without resample_spatial

#920

* actual resolution does not matter, will be resampled anyway

#920

* update main

#920

* debug

#920

* fix tests

#920

* quick fix temporal resolution of layer metadata

Depends on changes in openeo-geotrellis-extensions.

#920

* restore Orfeo implementation

#920

* restore SENTINEL3_SLSTR_L2_LST resolution

#920

* cleanup

#920

* DRY

#920

* adapt version and CHANGELOG #920
@bossie
Copy link
Collaborator Author

bossie commented Oct 30, 2024

Fixed by returning a data cube in EPSG:4326 and thereby provoking an actual call to resample_spatial (+ some fixes related to the hourly resolution of the Sentinel 3 collections).

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

@bossie bossie

Labels
None yet
Projects
None yet
Milestone
No milestone
1 participant
@bossie