search

oxcsml / epimap

MIT License
1 stars 0 forks source link

Fix Local authority mismatch issue #2

Closed ywteh closed 4 years ago

ywteh commented 4 years ago

Currently there are some mismatches.

upaq commented 4 years ago

Cornwall and Isles of Scilly in uk_cases.csv is Cornwall in metadata.csv.

Assuming that we would get updates of uk_cases.csv, I'm renaming in metadata.csv.

ywteh commented 4 years ago

uk_cases.csv is generated by process_uk_cases.py and meta.csv by process_metadata.py

Can we edit either/both to produce consistent lists of UTLA?

-yw

::: Yee Whye Teh ::: Professor of Statistical Machine Learning ::: ::: Oxford Statistics and DeepMind ::: http://mlcs.stats.ox.ac.uk/people/teh/ :::

On Tue, Jul 14, 2020 at 4:10 PM upaq notifications@github.com wrote:

Cornwall and Isles of Scilly in uk_cases.csv is Cornwall in metadata.csv.

Assuming that we would get updates of uk_cases.csv, I'm renaming in metadata.csv.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/rs-delve/Rmap/issues/2#issuecomment-658236481, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABKM6WPPS5RNSJOSXHM5JDR3RYNHANCNFSM4OWWH4KA .

upaq commented 4 years ago

Cornwall and Isles of Scilly are reported jointly, but exist as two separate UTLAs. For the purpose of plotting, easiest is to put the reporting under Cornwall (250k people) and not show Isles of Scilly (1k people)? I assume they're grouped together because of privacy.

14

ywteh commented 4 years ago

Ah good point, thanks, sounds good to me.

On Tue, 14 Jul 2020 at 5:29 pm, upaq notifications@github.com wrote:

Cornwall and Isles of Scilly are reported jointly, but exist as two separate UTLAs. For the purpose of plotting, easiest is to put the reporting under Cornwall (250k people) and not show Isles of Scilly (1k people)? I assume they're grouped together because of privacy.

14 https://github.com/rs-delve/Rmap/pull/14

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/rs-delve/Rmap/issues/2#issuecomment-658280667, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABKM6RS4NSREOTGJ6SKCZDR3SBWJANCNFSM4OWWH4KA .

-- ::: Yee Whye Teh ::: Professor of Statistical Machine Learning ::: ::: Oxford Statistics and DeepMind ::: http://mlcs.stats.ox.ac.uk/people/teh/ :::

upaq commented 4 years ago

Cornwall and Isles of Scilly are grouped together in UK reporting, but are two separate UTLAs.

Cornwall has 250k people, Isles of Scilly has 1k people. I assume they're grouped together because of privacy or practical reasons? Suggestion to take and plot reporting for both under the Cornwall UTLA in the map.

14

upaq commented 4 years ago

There is a cleaner solution, as Hackney and the City of London are also grouped together.

Scotland groups a few UTLAs together in an NHS trust and reports on trust level.

16 creates the right population / lat+longitude / population density for the larger "reporting regions". We model on the reporting regions (so Cornwall and Isles of Scilly together).

upaq commented 4 years ago

This is fixed in https://github.com/rs-delve/Rmap/blob/master/process_metadata.py