search

opensafely / SRO-Measures

1 stars 0 forks source link

SRO Report Improvements #53

Closed Jongmassey closed 2 years ago

Jongmassey commented 2 years ago

@brianmackenna has requested @LFISHER7 @Jongmassey and @tomodwyer work on improving the UI and UX of the SRO measures report

This issue will act as a master list of requirements, each of which may have their own issues.

Preliminary ideas:

It'd be really good to understand what the specific requirements are for this work before undertaking any development.

brianmackenna commented 2 years ago

Great stuff. I think there are two overall objectives. We should timebox this at 2 weeks to ensure we have a clear plan/options ready for the New Year.

Objective 1: Make the reports more visually appealing

I think static charts are sufficient for this time boxing. We previously implemented interactive charts but we removed them for a variety of reasons.

Objective 2: Implement in EMIS

The current report runs in TPP (this will require updating with latest months data). We should see how far we get within the time box to implement in EMIS.

Related to this is federated analysis - there are IG considerations with combining decile charts. Once we have running in EMIS we should convene wider discussion on this.

sebbacon commented 2 years ago

For the visual work, @tomodwyer will certainly be involved. AFAIK he has never set up or worked on notebooks here, so a dependency is to create a readme or similar that will help him get up an running with the minimum of fuss. Also perhaps a quick tour of the UI and code (maybe record a 5-10 min google video via Meet?)

LFISHER7 commented 2 years ago

@tomodwyer there's a quick run through of the SRO report here. There's also a list of changes I think are in scope in this doc. Would be good to get your thoughts next week on what you think the best approach would be

LFISHER7 commented 2 years ago

Leaving the interactivity element for another day, I think this is now all covered.