-
Notifications
You must be signed in to change notification settings - Fork 96
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature: biomass loss widget + woody biomass analysis #3664
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Widget is working well and looking good. Only exception is that we seem to have a bug on the view on map button.
Side note: I am also seeing a console bug on the climate-watch widget (historical emissions). This is an important one for the climate tab, and it seems like it is the same error on production and locally - we should fix that if we can, as it will also be under scrutiny on the Dec 5th event too.
:
http://localhost:5000/dashboards/country/BRA?category=climate
@benlaken you can now see the widget inside the map when analysing the area, so I would say this PR is complete. Can you review? |
We're having some bugs when trying to analyze a geometry or polygon for this layer (in the map). Seems like the widget never renders - just infinite spinner. |
@benlaken @blayhem updated! But I think the endpoint is failing to regularly for this to be usable. You have all the analysis types working now. Also the sentences no longer make sense as WDPA, custom areas etc dont really have nice names. |
@edbrett this lgtm - true the analysis fails often, but that is mostly out of our hands - I can possibly tweak the params of the micro-service to make it less accurate but more reliable, and try and find a balance. Seems to work well apart from that. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Working well 👍
Overview
Updates the old Climate emissions widget to add the following:
Additional features: