openforis / sepal

Geographical Data Processing in the Cloud
https://sepal.io/
MIT License
209 stars 46 forks source link

display of out of bounds date in alert change #228

Closed 12rambau closed 2 years ago

12rambau commented 2 years ago

If you display the dates from the change alert recipe, everything bigger than the end of monitoring dates is displayed in red which is very confusing. Don't you think it would worse the shot to mask them ?

Capture d’écran 2022-08-25 à 13 29 19
cdanielw commented 2 years ago

Maybe we could add an additional option in the layer visualization panel, to mask out-of-bound values? That option could be enabled by default for these layers.

12rambau commented 2 years ago

right it would be nice, but in this case I would more expect nodata value for places without alert both in the display and the export.

I didn't mentioned it in the issue message, but here I think the most urgent matter is the inversion of the bar and the bug in reading the values on the map (when you hover it's always nodata)

cdanielw commented 2 years ago

Resolved by https://github.com/openforis/sepal/commit/8e81047fdb27ec2143497ec39daba161b79a2110b