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

Add comment explaining phone counting strategy

Browse Source
This commit is contained in:
Chris Hill-Scott
2021-07-05 15:24:04 +01:00
parent 5a378fe51f
commit a0d6c153bf
1 changed files with 7 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
app/models/broadcast_message.py
View File

@@ -184,10 +184,17 @@ class BroadcastMessage(JSONModel):
estimated_area = self.simple_polygons.estimated_area estimated_area = self.simple_polygons.estimated_area
if estimated_area > ESTIMATED_AREA_OF_LARGEST_UK_COUNTY: if estimated_area > ESTIMATED_AREA_OF_LARGEST_UK_COUNTY:
# For large areas, use a naïve but computationally less
# expensive way of counting the number of phones in the
# bleed area
count = self.count_of_phones * ( count = self.count_of_phones * (
self.simple_polygons_with_bleed.estimated_area / estimated_area self.simple_polygons_with_bleed.estimated_area / estimated_area
) )
else: else:
# For smaller areas, where the computation can be done in
# a second or less (approximately) calculate the number of
# phones based on the ammount of overlap with areas for
# which we have population data
count = CustomBroadcastArea.from_polygon_objects( count = CustomBroadcastArea.from_polygon_objects(
self.simple_polygons_with_bleed self.simple_polygons_with_bleed
).count_of_phones ).count_of_phones