Geo refactor for Quidel covidtests

[krivard]

GeoMapping related work after talking to Dmitry:

• refactor the geo_map.py to make the steps for msa/hrr_to_parent_state independent from traditional geo mapping
• jingjing will talk with Roni next Friday about whether to stick with geographical pooling (considering parent states).

Originally posted by @jingjtang in #193 (comment)

[jingjtang]

Waiting for rf_geo_refactor to be merged into main

[krivard]

Geo refactor merged!

[jingjtang]

The code for switching to geo_map utils is ready, but problem encountered when running the unit tests. The unit test failed for a simple data frame which have integer values at zipcode level but float numbers after using the functions in geo utils to aggregate it from zipcode level to msa level. Will talk to @dshemetov first. If there does exist a problem in the utils, it will affect other signals that start from zip as well.

[jingjtang]

The code for switching to geo_map utils is ready, but problem encountered when running the unit tests. The unit test failed for a simple data frame which have integer values at zipcode level but float numbers after using the functions in geo utils to aggregate it from zipcode level to msa level. Will talk to @dshemetov first. If there does exist a problem in the utils, it will affect other signals that start from zip as well.

Talked to Dmitry already. This is not a problem in the geo utils. But switching to geo utils from the traditional version does affect the results to be reported. @krivard We need to decide a time point to re-upload all of the reports dating back to late May.

[krivard]

Let's try for uploading tomorrow afternoon. The meta cache update will push ingestion to tomorrow evening, and if there's anything to debug we can catch it in the morning.

[chinandrew]

Is this resolved?

[krivard]

Yep! 🧹