Import police data for EVAW hub #702
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
struan
force-pushed
the
680-vawg-crime-data-import
branch
from
0d0e55e to
b6762a6
Compare
struan
force-pushed
the
680-vawg-crime-data-import
branch
from
b6762a6 to
fab9345
Compare
zarino
reviewed
Nov 10, 2025
Member
zarino
left a comment
zarino
left a comment
There was a problem hiding this comment.
Couple of suggestions re: wording, as requested!
conf/imports.json
Outdated
| }, | ||
| { | ||
| "name":"reported_rapes_per_100k", | ||
| "label":"Reported Rapes per 100k", |
Member
There was a problem hiding this comment.
“Reported incidents of rape, per 100,000 people” perhaps?
conf/imports.json
Outdated
| }, | ||
| { | ||
| "name":"reported_rapes", | ||
| "label":"Reported Rapes", |
Member
There was a problem hiding this comment.
“Total reported incidents of rape” perhaps?
| "subcategory":null, | ||
| "release_date":"July 2025", | ||
| "source_label":"UK and Scottish governments, combined by mySociety", | ||
| "source":"https://www.gov.uk/government/statistical-data-sets/police-recorded-crime-and-outcomes-open-data-tables, https://www.gov.scot/collections/recorded-crime-in-scotland/", |
Member
There was a problem hiding this comment.
A minor thing, but I think this will cause an issue in the Area and Sources templates, where they expect the dataset’s source url to be a single URL which can be put inside the href attribute of a single link. If we want datasets with multiple sources (good idea!) we will need to update the template and/or view to handle that.
Member
Author
There was a problem hiding this comment.
oh, yes, yes it will. I will leave it as now and then we can sort it out later, if that's ok?
Member
There was a problem hiding this comment.
Yep! Can you ticket up, so we don’t forget?
struan
force-pushed
the
680-vawg-crime-data-import
branch
from
fab9345 to
82aefc3
Compare
it falls over on na values also do fill blanks first so that they don't get dropped
This doesn't fix the JS that's doing this
We're starting to add data which is the same value over a number of years so best to mark the explicitly so we can display it sensibly
Does this as a line graph, with a plugin to sort out avoiding label/legend overlap because that's the only way. Also make sure the minimum value on the x axis is always 0. We can hard code this because it's also fine for the horizontal bar charts we have
Make sure it splits it out a la range data
If you use both you end up replacing data and as the DIS councils don't cover everything you end up with bad data, so only use STC
adds a filter option to import from config that lets you only import rows where the confuigured column matches the specified value
includes a generator script that gathers all the data and turns it into an easy to import format
Also remove a duplicated `is_range` key in imports.json and provide a better source URL for the safeguarding datasets.
- Make it clear that this is a recommended _minimum_ - Round up to nearest whole number (you can’t have half a refuge centre)
struan
force-pushed
the
680-vawg-crime-data-import
branch
from
e10e717 to
cdf0761
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #680