Open-EO / openeo-web-editor

An interactive and easy to use web-based editor for the OpenEO API.
https://editor.openeo.org
Apache License 2.0
12 stars 17 forks source link

resample_spatial parameters bug #274

Closed clausmichele closed 2 years ago

clausmichele commented 2 years ago

When inserting the resample_spatial process, if you edit the parameters you would automatically get a projection selected in the respective field. However, I guess it should be set to null as default, since it's not a mandatory parameter.

image

m-mohr commented 2 years ago

Thanks, will check. Which back-end is this for?

clausmichele commented 2 years ago

Well, I'm using the Eurac back-end (currently implementing resample_spatial) but I guess it's the same for the others.

m-mohr commented 2 years ago

Looking at https://processes.openeo.org/#resample_spatial I see that 0 is the default value (means: no change) so the behavior is correct, right?

clausmichele commented 2 years ago

Yes, the default for the projection is null, but if you open the pop-up to modify the process parameters it automatically select the EPSG:2000

m-mohr commented 2 years ago

Sorry, my bad, I looked at resolution instead of projection. Will be fixed.

m-mohr commented 2 years ago

Can't reproduce on VITO: Are you on the latest version of the Web Editor? Is your schema correctly returned from the back-end? It might be that "null" as default value get's removed by your back-end as it thinks null means "don't serialize to JSON", I've seen that before in the Java implementation.

clausmichele commented 2 years ago

Found the problem, the resample_spatial process in our back-end is an outdated version. So the bug it's on our side not on the Web editor sorry! https://github.com/Open-EO/openeo-spring-driver/issues/65