The-Strategy-Unit / The-Strategy-Unit/nhp_inputs

Show message in procedures/mitigators boxes if there's insufficient data to show

Open
#401 7 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

should techdebt
Dominant language
R
Stars
2
Forks
1
Avg merge
15h 39m
Merged PRs (30d)
6

Description

If there's no top-6 data to show in the procedures or diagnoses tables in each mitigators page, it shows NaN. Perhaps best to check for lack of data and supply a validate-need message saying there's insufficient data to show line items.

Example: Airedale, OP-EF-001.

Similarly, the age-sex pyramid will not render if there's no data. Consider showing a message instead.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reproducing the missing-data case for the procedures and diagnoses tables on a mitigators page, using the Airedale OP-EF-001 example. Then inspect the age-sex pyramid rendering path and the existing validate-need messages. Done means each affected view shows an insufficient-data message instead of NaN or a blank/non-rendering chart.

Written by the indexing model from the issue text.

Assessment

Tech stack
r
Domain
data-visualization
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.