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

no results for bbox near date line #279

Open
bossie opened this issue Apr 3, 2024 · 7 comments · May be fixed by #310
Open

no results for bbox near date line #279

bossie opened this issue Apr 3, 2024 · 7 comments · May be fixed by #310
Assignees
@EmileSonneveld
Labels
bug Something isn't working

Comments

@bossie
Copy link
Collaborator

bossie commented Apr 3, 2024

This process graph will return an empty GeoTiff for sync requests and no assets for batch jobs:

{
  "process_graph": {
    "loadcollection1": {
      "process_id": "load_collection",
      "arguments": {
        "bands": [
          "B02",
          "B03",
          "B04",
          "B08"
        ],
        "id": "SENTINEL2_L2A",
        "spatial_extent": {
          "west": 178.7384,
          "east": 178.8548,
          "south": 70.769,
          "north": 70.8254
        },
        "temporal_extent": [
          "2024-04-02",
          "2024-04-03"
        ]
      }
    },
    "saveresult1": {
      "process_id": "save_result",
      "arguments": {
        "data": {
          "from_node": "loadcollection1"
        },
        "format": "GTiff",
        "options": {}
      },
      "result": true
    }
  }
}

Internal ref: CDSE-96
@bossie bossie added the bug Something isn't working label Apr 3, 2024
bossie added a commit that referenced this issue Apr 3, 2024
@bossie
replace println with logger #279
f8e9832
@bossie
Copy link
Collaborator Author

bossie commented Apr 3, 2024

FWIW, forcing the CRS to EPSG:32601 makes it work.

@JeroenVerstraelen
Copy link
Contributor

JeroenVerstraelen commented Apr 9, 2024 •
edited
Loading

Has to be fixed in the next sprint. This was originally an issue from CDSE Zendesk.

@EmileSonneveld
Copy link
Contributor

The PG gives a result on https://openeo-dev.vito.be
But indeed no result on https://openeo-staging.creo.vito.be (no errors, no files)

@EmileSonneveld
Copy link
Contributor

This might be due the gaps in S2 data and not the date line.
When enlarging the extent I got some data crossing the line.
image

#280

@EmileSonneveld
Copy link
Contributor

Sentinel 5 also had missing data close to the date line. Found by student via Victor Verhaert

EmileSonneveld added a commit that referenced this issue May 28, 2024
@EmileSonneveld
Add testMissingS2DateLine #279
bfbe212
EmileSonneveld added a commit that referenced this issue Jun 10, 2024
@EmileSonneveld
Avoid north pole impricision when reprojecting between LatLng and UTM…
5eec0bf 
…. Needs more testing. #279
EmileSonneveld added a commit that referenced this issue Jun 27, 2024
@EmileSonneveld
Fix tests. #279
0daedc7
@EmileSonneveld EmileSonneveld linked a pull request Jun 28, 2024 that will close this issue
Dateline #310
Open
EmileSonneveld added a commit that referenced this issue Jun 28, 2024
@EmileSonneveld
Fix to make tests run on Jenkins. #279
d04fcf4
EmileSonneveld added a commit that referenced this issue Jun 28, 2024
@EmileSonneveld
Fix to make tests run on Jenkins. #279
93c2b75
EmileSonneveld added a commit that referenced this issue Jul 23, 2024
@EmileSonneveld
Assert sensible extends in UTM zones. #279
223b579
EmileSonneveld added a commit that referenced this issue Jul 24, 2024
@EmileSonneveld
Better choose common CRS. Add test for UTM->LatLng and LatLng->UTM. A…
7b580ec 
…dd health check for ProjectedExtends. #279
@jdries
Copy link
Contributor

jdries commented Sep 9, 2024

I logged a downstream issue that could also help here:
locationtech/proj4j#113

EmileSonneveld added a commit that referenced this issue Sep 17, 2024
@EmileSonneveld
Test EPSG:3035 and EPSG:4326 too. #279
0309ef2
@EmileSonneveld
Copy link
Contributor

TODO: Change code so that in unexpected cases, the old behaviour is used

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

@EmileSonneveld EmileSonneveld

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Dateline Open-EO/openeo-geotrellis-extensions
4 participants
@bossie @EmileSonneveld @JeroenVerstraelen @jdries