search

wri / gfw_forest_loss_geotrellis

Global Tree Cover Loss Analysis using Geotrellis and SPARK
MIT License
10 stars 8 forks source link

Replace glad with integrated alerts in gfwpro_dashboard. #201

Closed danscales closed 11 months ago

danscales commented 11 months ago

Replace glad with integrated alerts in gfwpro_dashboard.

Many of these changes come from Justin's original prototype, including the switch to the 10x10m grid.

Added a new DataConfLevelsLayer that decodes three levels of confidence. The other confidence layer DataConfLayer only decodes two levels and returns a Boolean. So, I added a new layer, so that the existing code using DataConfLayer that expects a boolean doesn't all have to change.

In order to reduce the number of things that are changing, we are intentionally keeping the glad* column names in the output, rather than switching to integrated*. We will update those in a later change, such as when we add in confidence levels in the output.

jterry64 commented 11 months ago

Oh one more note @danscales - this scale is 10 times larger than GLAD, so you should do some performance testing in develop to see if our current Pro dashboard scaling works for this (for both single list and batch).

danscales commented 11 months ago

Oh one more note @danscales - this scale is 10 times larger than GLAD, so you should do some performance testing in develop to see if our current Pro dashboard scaling works for this (for both single list and batch)

Thanks for your comments, Justin! I have reminded Kinshuk that this will be an instant switch for all customers to integrated alerts. I've also told him that it will be more expensive than glad alerts. I will check in now and create a JAR that I can use for testing (on develop only at first) how expensive the runs are, and whether dashboard scaling is working well.