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

Add sub-headings to group the mitigators #109

Open
jacgrout opened this issue Dec 13, 2024 · 4 comments
Open

Add sub-headings to group the mitigators #109

jacgrout opened this issue Dec 13, 2024 · 4 comments
Labels
could MoSCoW priority enhancement New feature or request
Milestone

Comments

@jacgrout
Copy link
Member

Sub-headings to separate out in the presentation beween

Accident & Emergency
Inpatients
Outpatients

as well as

Admission Avoidance
Efficiencies

@jacgrout jacgrout added this to the 0.8.0 milestone Dec 13, 2024
@matt-dray
Copy link
Contributor

Thanks Jac. Just to confirm, I think you mean to group up the charts like this (pointrange example)?

## Inpatients

### Admission avoidance

[Faceted plot of schemes/mitigators]

### Efficiencies

[Faceted plot of schemes/mitigators]

## Outpatients

etc

And presumably ignoring headers/subheaders if the user didn't select any mitigators under those headings.

Or did you mean to group up the mitigator selection options in the left-hand settings panel? (Being looked into as part of #43 and subsequently #98.)

@matt-dray matt-dray added the enhancement New feature or request label Dec 17, 2024
@matt-dray matt-dray added the could MoSCoW priority label Jan 14, 2025
@matt-dray matt-dray modified the milestones: v0.7.0, Backlog Jan 14, 2025
@matt-dray
Copy link
Contributor

On reflection, I think this issue was meant more in the context of the heatmaps, specifically (but I suspect it applies throughout the app).

There'd be some jiggery-pokery required, of course, as we'd be going from one to multiple plots. These sections would have to appear or be ignored conditionally on the user's inputs. I could imagine splitting the underlying dataset into a list of dataframes that's converted to a list of named plots that could then be spat out into the UI. NULL elements could be ignored and not plotted, nor have their sub-heading printed.

@craig-parylo and I chatted about some other (less complicated) solutions like adding an extra axis title that groups the mitigators. I thought {legendry} had a function for this, but I think it applies to continuous axes only.

@jacgrout
Copy link
Member Author

Yes it was meant in the context of just the heatmaps.

@matt-dray
Copy link
Contributor

This is something that would make particular sense in the context of #124.

@matt-dray matt-dray modified the milestones: v0.7.1, v0.7.2 Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
could MoSCoW priority enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants