-
Notifications
You must be signed in to change notification settings - Fork 2
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
Ethnicity treemap error #586
Comments
Merged
#587 didn't fix this, it's still happening |
I have fixed in the other branch
…On Tue, 11 Feb 2025, 14:03 Michael Barton, ***@***.***> wrote:
#587
<#587>
didn't fix this, it's still happening
—
Reply to this email directly, view it on GitHub
<#586 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAICOV3VEMPA5EORZGIF4BD2PH7I7AVCNFSM6AAAAABW3SDRKWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMNJQHEYTMMJYHE>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***
.com>
[image: mbarton]*mbarton* left a comment
(rcpch/national-paediatric-diabetes-audit#586)
<#586 (comment)>
#587
<#587>
didn't fix this, it's still happening
—
Reply to this email directly, view it on GitHub
<#586 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAICOV3VEMPA5EORZGIF4BD2PH7I7AVCNFSM6AAAAABW3SDRKWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMNJQHEYTMMJYHE>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***
.com>
|
@eatyourpeas ah ok, which branch? Should I merge #591 or will that conflict with you? |
It is in the PR about patient validation. Final commit. My only worry is the impact that PR is having on the tests |
OK cool looks like we fixed it in pretty much the same way. I’ll merge my PR just to save my inbox as I get an email on every error at the moment |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Happens if you haven't uploaded any data yet
The text was updated successfully, but these errors were encountered: