Skip to content

support geo type mapping resolutions #542

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

Open
sgratzl opened this issue May 14, 2021 · 2 comments
Open

support geo type mapping resolutions #542

sgratzl opened this issue May 14, 2021 · 2 comments

Comments

@sgratzl
Copy link
Member

sgratzl commented May 14, 2021

coming from #389

e.g. query all counties of the state of California

possible mappings:

  • nation -> state
  • state -> county
  • hrr -> state
@krivard
Copy link
Contributor

krivard commented May 17, 2021

this might be more easily implemented in delphi_utils.geomap, possibly as an extension of /variation on get_geo_values():

https://github.com/cmu-delphi/covidcast-indicators/blob/261b3a3868abd1527d8ac9867a8aca78f8ef46eb/_delphi_utils_python/delphi_utils/geomap.py#L559-L576

@krivard
Copy link
Contributor

krivard commented Aug 6, 2021

Now that cmu-delphi/covidcast-indicators#1159 has been merged, the following has to happen:

  • @korlaxxalrok to pull the release lever in covidcast-indicators (no earlier than Monday 9 Aug)
  • design a syntax for specifying contained/child regions as a URL parameter of a covidcast request
  • implement in the API server by decoding the new syntax and applying delphi_utils.geomap.GeoMapper.get_geos_within()

I don't have any huge preferences on syntax, so long as we avoid the possibility where the same geo_id parameter value could be interpreted as multiple geo types. @sgratzl may have guidance?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants