cmu-delphi / covidcast-indicators

Back end for producing indicators and loading them into the COVIDcast API.
https://cmu-delphi.github.io/delphi-epidata/api/covidcast.html
MIT License
12 stars 17 forks source link

Refactor weighted aggregation for hhs as a method in the geomapper #2038

Open minhkhul opened 2 months ago

minhkhul commented 2 months ago

This will definitely work, though it would be better if this were implemented as a method in the geomapper itself, for 2 reasons: reuse, and speed. aggregate_by_weighted_sum has to create the weights at every single time_value, which means it's definitely slower.

Originally posted by @dsweber2 in https://github.com/cmu-delphi/covidcast-indicators/pull/2036#pullrequestreview-2263775663