-
Notifications
You must be signed in to change notification settings - Fork 16
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 a "KPI" subpage of the "statistics" menu item #180
Comments
Hah, make it "KPI and CTS" and base the calculation only on the info present in the system (and warn about this). Possibly reference #14 . List B-centres (a subset of the KPI centres) that are listed in the SIS, and explain the CTS reqs. |
In some ways, this issue is a blocker for #22 . |
I have prepared an extra page, but I think it may require additional functions and/or somewhat different logic, in some cases, to save on potentially costly queries and just keep their results for multiple uses -- if that makes sense, and only @margaretha can assess that :-) |
Meeting decision: make KPI #3 less prominent, just mention it at the bottom. Eliza @margaretha will take over for the data access functions. Commit forthcoming. |
It's in your hands now, @margaretha -- thanks in advance :-) |
Closing with thanks :-) |
As above -- this has to stand out.
It would be good to get the logic behind the calculation established and cite it precisely on the page, as well.
I consider this an urgent task.
The text was updated successfully, but these errors were encountered: