Allow user to select specific sites, rather than whole provider

Open
#212 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
5/5
Estimated time
Over a week
Newbie friendliness
25/100
Issue type
Feature
Clarity
Needs clarification
Activity status
Stale
Tech stack
r
Domain
data, frontend

Research direction

Start by reviewing the home page of the inputs app and how charts throughout the app consume provider and hospital-site selections. Define what site inclusion or exclusion should do, including its effect on peer selection, and confirm the required follow-up work in the model and outputs repositories before considering the feature complete.

Written by the indexing model from the issue text.

Description

could question

Currently, the model runs for all hospital sites for a provider. Outputs are aggregated at the level of hospital site (apart from A&E).

It may be useful to run the model for different sites with different assumptions. While there is nothing ng you from running the model for a provider but only ever using the results for a specific site, it may give people more reassurance to be able to run the model for specific sites.

We could add a selection on the home page of the inputs app to select which of a provider's sites to include or exclude.

Adding in this option though would require us to also filter charts throughout the app. This may also invalidate the peer selection.

If this is something that we want to implement, then additional issues need to be created in the model repo and the outputs repo.

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

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.

More from The-Strategy-Unit/nhp_inputs

All issues in The-Strategy-Unit/nhp_inputs

Similar issues

More R issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.