Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update mappings (#20) #28

Merged
merged 14 commits into from
Sep 22, 2022
Merged

Update mappings (#20) #28

merged 14 commits into from
Sep 22, 2022

Conversation

sophiemeakin
Copy link
Collaborator

Update LTLA- and UTLA-Trust mappings with new data.

To preserve access to the old mapping (based on SUS data until 30 September 2020), we have also changed the way that the mappings are accessed. Instead of accessing the data directly, the mappings can be accessed by load_mapping(scale, source), where scale defines the geographical scale (either “utla” or “ltla”), and source defines the data source on which the mapping is based (either “sus” for the mapping derived from Secondary Uses Service (SUS) healthcare data, or “link” for the mapping derived from linked COVID-19 cases and admissions).

Both vignettes (mapping creation and mapping overview) have been updated to reflect these changes.

* Script to make four mappings: two scales (LTLA-Trust and UTLA-Trust) and two data sources (HES and linked cases-admissions) - this was previously .Rmd file.
* Save resulting mappings in a single internal dataset, mappings; replaces trust_ltla_mapping and trust_utla_mapping.
* level -> map_level
* source -> map_source
* add summary of making the new mapping
* small edits to mapping based on SUS data
* remove code chunks - this now lives in data-raw/make_mapping.R and is referenced in the vignette
* not specific to either mapping
* drop details of methods (in mapping creation vignette)
@sophiemeakin sophiemeakin requested a review from sbfnk August 23, 2022 15:36
@sophiemeakin sophiemeakin changed the title Update mappings Update mappings (#20) Aug 23, 2022
@sophiemeakin sophiemeakin marked this pull request as ready for review September 22, 2022 11:08
@sophiemeakin
Copy link
Collaborator Author

Discussed changes offline with @sbfnk, merging.

@sophiemeakin sophiemeakin merged commit c5d3a0f into main Sep 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Not all NHS trusts from the NHS API are included in trust_ltla_mapping
1 participant