VAWG Data Hub preview - #733
Open
zarino wants to merge 24 commits into
Open
Conversation
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)
This was referenced Jul 23, 2026
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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 #700.