Open-EO / openeo-earthengine-driver

openEO back-end driver for Google Earth Engine.
Apache License 2.0
21 stars 7 forks source link

Server error: Invalid GeoJSON geometry #53

Closed jneijt closed 4 years ago

jneijt commented 4 years ago

I'm getting these 500 internal server errors when running our process graphs using a GeoJSON:

Server  error: Invalid GeoJSON geometry: 
{
  "geodesic": false,
  "type": "Polygon",
  "coordinates": [
    [
      [0, 0], [null, 0], [null, null], [0, null], [0, 0]
    ]
  ],
  "crs": {
    "type": "name",
    "properties": {
      "name": "EPSG:4326"
    }
  }
}

The process graph used:

{
  "process_graph": {
    "dc": {
      "process_id": "load_collection",
      "arguments": {
        "id": "COPERNICUS/S2_SR",
        "spatial_extent": {
          "type": "Polygon",
          "coordinates": [
            [
              [
                7.56926566943041,
                47.43044841749601
              ],
              [
                7.567760272677281,
                47.42486788741891
              ],
              [
                7.57045962027453,
                47.42032030185089
              ],
              [
                7.57768162600121,
                47.41777708806601
              ],
              [
                7.57960765888614,
                47.41580074117329
              ],
              [
                7.579608823537835,
                47.41580056664936
              ],
              [
                7.57960998818957,
                47.41580039212543
              ],
              [
                7.579612317493041,
                47.41580004307757
              ],
              [
                7.579616976099982,
                47.41579934498185
              ],
              [
                7.5796262933137815,
                47.41579794879038
              ],
              [
                7.5796449277414615,
                47.41579515640733
              ],
              [
                7.58378343256859,
                47.41158061676611
              ],
              [
                7.5877406910582,
                47.41156566207061
              ],
              [
                7.58811582752084,
                47.4135566053398
              ],
              [
                7.603614910275401,
                47.41855457409649
              ],
              [
                7.611730380383661,
                47.4232039939073
              ],
              [
                7.617390305662789,
                47.428660483137705
              ],
              [
                7.607422778933251,
                47.43266457934549
              ],
              [
                7.58490677583138,
                47.428623716235194
              ],
              [
                7.581206884670681,
                47.42878068945941
              ],
              [
                7.58010025007217,
                47.4320482150913
              ],
              [
                7.57321071083215,
                47.43641037283351
              ],
              [
                7.5689061636537,
                47.43590859148939
              ],
              [
                7.56926566943041,
                47.43044841749601
              ]
            ]
          ]
        },
        "temporal_extent": [
          "2020-08-09",
          "2020-08-19"
        ],
        "bands": [
          "TCI_R",
          "TCI_G",
          "TCI_B"
        ]
      }
    },
    "reduce": {
      "process_id": "reduce_dimension",
      "arguments": {
        "data": {
          "from_node": "dc"
        },
        "reducer": {
          "process_graph": {
            "min": {
              "arguments": {
                "data": {
                  "from_parameter": "data"
                }
              },
              "process_id": "min",
              "result": true
            }
          }
        },
        "dimension": "t"
      }
    },
    "apply": {
      "process_id": "apply",
      "arguments": {
        "data": {
          "from_node": "reduce"
        },
        "process": {
          "process_graph": {
            "2": {
              "process_id": "linear_scale_range",
              "arguments": {
                "x": {
                  "from_parameter": "x"
                },
                "inputMin": 0,
                "inputMax": 255,
                "outputMax": 255
              },
              "result": true
            }
          }
        }
      }
    },
    "save": {
      "process_id": "save_result",
      "arguments": {
        "data": {
          "from_node": "apply"
        },
        "format": "PNG",
        "options": {
          "red": "TCI_R",
          "blue": "TCI_B",
          "green": "TCI_G"
        }
      },
      "result": true
    }
  }
}

Did something change in the way the GeoJSON is expected by the backend, or is this just a temporary issue due to the active development?

Thanks!

m-mohr commented 4 years ago

That looks like an issue with the recent changes we did to the projections. Likely needs a bug fix...

m-mohr commented 4 years ago

@jneijt This should be fixed now.

jneijt commented 4 years ago

Yep, just tested it and I can confirm this is now fixed 👍 Thanks!