search

AI-SDC / SACRO-Viewer

A tool for fast, secure and effective output checking, which can work in any TRE.
Other
2 stars 1 forks source link

Display risk appetite #168

Closed benbc closed 1 year ago

benbc commented 1 year ago

ACRO can be parameterized according to the risk appetite of the TRE. The settings in operation when it runs are written to config.json in the outputs directory. We should display the settings (a series of name/value pairs) so the output checker can understand the intended risk appetite. These settings are quite verbose, so they should probably be in a collapsed element in the output detail pane.

benbc commented 1 year ago

There is a PR to take the new version of ACRO here, which this should build on.

lucyb commented 1 year ago

This change isn't in v0.3.0 unfortunately

bloodearnest commented 1 year ago

I'm gonna do the first part of this, render the config to html in the page. @tomodwyer can add styling/UX

bloodearnest commented 1 year ago

https://github.com/opensafely-core/sacro/pull/214

benbc commented 1 year ago

Can we aim to get that PR merged with a preliminary Simon UI, then put Tom's enhancements into a separate PR (but still under this ticket, I guess)?

bloodearnest commented 1 year ago

We can, but its pretty ugly atm

tomodwyer commented 1 year ago

Closed by #214