darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-08-18 21:49:37 -04:00
Code Issues Packages Projects Releases Wiki Activity

Bump utils to bring in new polygon simplification

Browse Source 
We’ve changed our simplification a bit so:
- polygons have slightly more points (see https://github.com/alphagov/notifications-utils/pull/873)
- the individual points have less precision (see https://github.com/alphagov/notifications-utils/pull/872)

Overall this reduces the size of the data we’re storing from 74MB to
63MB, and should make any pages where we are rendering lots of
coordinates load a bit quicker.
This commit is contained in:
Chris Hill-Scott
2021-06-24 12:27:12 +01:00
parent 416b5c3e26
commit e4ca78634d
6 changed files with 22 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
tests/app/main/views/test_broadcast.py
View File

@@ -811,8 +811,8 @@ def test_broadcast_page(
'England remove',
'Scotland remove',
], [
'An area of 177,439.8 square miles Will get the alert',
'An extra area of 6,392.3 square miles is Likely to get the alert',
'An area of 166,426.2 square miles Will get the alert',
'An extra area of 7,942.4 square miles is Likely to get the alert',
'40,000,000 phones estimated',
]),
([
@@ -837,7 +837,7 @@ def test_broadcast_page(
], [
'Islington remove',
], [
'An area of 9.7 square miles Will get the alert',
'An area of 9.6 square miles Will get the alert',
'An extra area of 4.7 square miles is Likely to get the alert',
'200,000 to 500,000 phones',
]),
@@ -908,8 +908,8 @@ def test_preview_broadcast_areas_page(
[[7, 8], [9, 10], [11, 12]],
],
[
'An area of 722.3 square miles Will get the alert',
'An extra area of 1,498.5 square miles is Likely to get the alert',
'An area of 543.3 square miles Will get the alert',
'An extra area of 1,595.9 square miles is Likely to get the alert',
'Unknown number of phones',
]
),
@@ -924,8 +924,8 @@ def test_preview_broadcast_areas_page(
(
[SKYE],
[
'An area of 3,205.0 square miles Will get the alert',
'An extra area of 763.4 square miles is Likely to get the alert',
'An area of 3,201.7 square miles Will get the alert',
'An extra area of 767.0 square miles is Likely to get the alert',
'4,000 phones estimated',
]
),

2
tests/app/models/test_broadcast_message.py
View File

@@ -34,7 +34,7 @@ def test_simple_polygons(fake_uuid):
# Because the areas are close to each other, the simplification
# and unioning process results in a single polygon with fewer
# total coordinates
[51],
[54],
]