Skip to content

Commit b21252f

Browse files
authored
Merge pull request #322 from cmu-delphi/geo_refactor_claimshosp
Refactor claims_hosp to use new geomapper functions
2 parents 317a4d5 + 707535e commit b21252f

File tree

2 files changed

+13
-33106
lines changed

2 files changed

+13
-33106
lines changed

claims_hosp/delphi_claims_hosp/update_indicator.py

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -96,13 +96,21 @@ def geo_reindex(self, data):
9696
"""
9797
geo_map = GeoMapper()
9898
if self.geo == "county":
99-
data_frame = geo_map.county_to_megacounty(
100-
data, Config.MIN_DEN, Config.MAX_BACKWARDS_PAD_LENGTH,
101-
thr_col="den", mega_col=self.geo)
99+
data_frame = geo_map.fips_to_megacounty(data,
100+
Config.MIN_DEN,
101+
Config.MAX_BACKWARDS_PAD_LENGTH,
102+
thr_col="den",
103+
mega_col=self.geo)
102104
elif self.geo == "state":
103-
data_frame = geo_map.county_to_state(data, state_id_col=self.geo)
105+
data_frame = geo_map.replace_geocode(data,
106+
from_code="fips",
107+
new_col=self.geo,
108+
new_code="state_id")
109+
data_frame[self.geo] = data_frame[self.geo]
104110
elif self.geo == "msa":
105-
data_frame = geo_map.county_to_msa(data, msa_col=self.geo)
111+
data_frame = geo_map.replace_geocode(data,
112+
from_code="fips",
113+
new_code=self.geo)
106114
elif self.geo == "hrr":
107115
data_frame = data # data is already adjusted in aggregation step above
108116
else:
@@ -119,7 +127,6 @@ def geo_reindex(self, data):
119127
assert (
120128
len(multiindex) <= (GeoConstants.MAX_GEO[self.geo] * len(self.fit_dates))
121129
), "more loc-date pairs than maximum number of geographies x number of dates"
122-
123130
# fill dataframe with missing dates using 0
124131
data_frame = data_frame.reindex(multiindex, fill_value=0)
125132
data_frame.fillna(0, inplace=True)

0 commit comments

Comments
 (0)