search

Open-EO / openeo-earthengine-driver

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

Swap bands B4 and B8, because non-sensical NDVI result #32

Closed glemoine62 closed 4 years ago

glemoine62 commented 4 years ago

You need to swap B4 and B8 (B8 should be b1, B4 should be b2) because the normallized difference is calculated as (b1-b2)/(b1+2).

m-mohr commented 4 years ago

Hi @glemoine62,

thanks for reporting. You are only referring to the examples in the README, right? Indeed, the example is incorrect. I'll update it.

glemoine62 commented 4 years ago

Yes, correct. I parsed the JavaScript result. However, it logically follows from the process graph.

m-mohr commented 4 years ago

Thanks, I fixed that.

Note that this example is not very performant and will change soon as we have found better and more performant ways to implement this with the new API version. I'll update example and implementation in the next weeks.