darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-06-24 01:11:15 -04:00
Code Issues Packages Projects Releases Wiki Activity

Rename variable to be clearer

Browse Source 
Better name than `population`, and
`smartphone_ownership_for_area_by_age_range` matches with
`SMARTPHONE_OWNERSHIP_BY_AGE_RANGE`
This commit is contained in:
Chris Hill-Scott
2020-09-16 08:20:37 +01:00
parent 8ea3f0141c
commit ce35200453
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
app/broadcast_areas/create-broadcast-areas-db.py
View File

@@ -80,18 +80,18 @@ def estimate_number_of_smartphones_in_area(country_or_ward_code):
raise ValueError(f'No population data for {country_or_ward_code}')
population = area_to_population_mapping[country_or_ward_code]
population_by_range = {}
smartphone_ownership_for_area_by_age_range = {}
for range, ownership in SMARTPHONE_OWNERSHIP_BY_AGE_RANGE.items():
min, max = range
population_by_range[range] = sum(
smartphone_ownership_for_area_by_age_range[range] = sum(
people
for age, people in population
if min <= age <= max
) * ownership
total_population = sum(dict(population).values())
total_phones = sum(population_by_range.values())
total_phones = sum(smartphone_ownership_for_area_by_age_range.values())
print( # noqa: T001
f' Population:{total_population: 11,.0f}'