Skip to content
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

Display raster median in tooltips #64

Closed
ericboucher opened this issue May 26, 2020 · 6 comments
Closed

Display raster median in tooltips #64

ericboucher opened this issue May 26, 2020 · 6 comments
Labels
enhancement New feature or request skills:React

Comments

@ericboucher
Copy link
Collaborator

When clicking on an admin boundary when a WMS raster is selected, I would like the median value to be displayed in the tooltip.

@ericboucher ericboucher added enhancement New feature or request skills:React labels May 26, 2020
@bencpeters
Copy link
Collaborator

bencpeters commented May 27, 2020

Implementation note:

2 ways to do this:

  1. We implement WCS layers as suggested in a comment in Surface temperature returned from Data Cube needs to be converted to celsius #43.
    This gives us the most flexibility to do data transforms, manipulations, define legends dynamically on the fly, etc. This will make the load time more similar to Impact layers though.

  2. We could dynamically fetch data just for the district on click. This has the advantage of doing the loading only on demand, and a much smaller memory/load time footprint (since we only have to load a relatively small district's worth of data), but it would mean a lot more individual requests to the geo servers if this feature was used a lot, and it would mean that the tooltip would have a bit of loading delay.

I think the second option is probably the better way to go for now though.

@bencpeters
Copy link
Collaborator

bencpeters commented Jun 18, 2020

Note: with the new API in #80, we probably want to implement this with an API call, rather than either 1) or 2) for performance reasons.

@ericboucher
Copy link
Collaborator Author

Should we add an id filter on the API, so that we can calculate this info for a specific zone?

@wadhwamatic
Copy link
Member

A reminder that we also need to display raster stats in a table view as well, not just in the tooltip. The table view would show a value for each admin area

@ericboucher
Copy link
Collaborator Author

@bencpeters @dubwalla we could trigger the zonal stats for rasters that have a WCS config in the background.

This behavior could be "shut off" either through the config.json or also through a toggle in the frontend?

Noting that the faster the API will be, the more seamless this would become

@wadhwamatic
Copy link
Member

Covered more practically with the options we have immediately available here: #1150

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request skills:React
Projects
None yet
Development

No branches or pull requests

3 participants