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

QE King's Lynn #907

Closed
eatyourpeas opened this issue Apr 25, 2024 · 3 comments · May be fixed by #933
Closed

QE King's Lynn #907

eatyourpeas opened this issue Apr 25, 2024 · 3 comments · May be fixed by #933
Assignees
Labels
bug Something isn't working

Comments

@eatyourpeas
Copy link
Member

There seems to be an error with King's Lynn specifically where aggregations are counting all children in the organisation, at ICB/NHS region and national levels, but not at Trust level. It looks like they are being filtered out.

@mbarton
Copy link
Member

mbarton commented Apr 29, 2024

[posting summary from a call with Simon, Danny and Michael 29/04/24]

We think the issue is probably that now we have added additional filtering, no cases from QE Kings Lynn should be included in the aggregations. However an organisation aggregation row was added previously and our code at the moment doesn't reset it again to zero, it simply skips over any abstraction level that has no corresponding cases.

@eatyourpeas will try changing the code to always write at all abstraction levels every time. We believe that to be safe as even if we have bugs in our aggregation code it can always be derived from the underlying data - so it's fine to overwrite a previous aggregation row even if there's no corresponding data for the organisation now our additional filtering applies

@mbarton
Copy link
Member

mbarton commented May 1, 2024

@dc2007git and I looked at this today. We removed all KPI metrics so they would be recalculated the next time aggregations were run.

BaseKPIMetrics.objects.all().delete()

This removed any outdated rows but unfortunately it does not seem to have fixed this issue.

The Trust column that was blank before is now missing entirely. The organisation shows zero cases at the top level but then does show cases in the rows below. It also shows a "Local health board" column which should not apply here as we're not in Wales.

@mbarton
Copy link
Member

mbarton commented May 2, 2024

This is fixed, we’ll track everything else under other issues

@mbarton mbarton closed this as completed May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
2 participants