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.

Sign up for GitHub

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

Jump to bottom

Urinary Albumin Level (ACR) validation rules are wrong? #578

Closed
mbarton opened this issue Feb 9, 2025 · 2 comments · Fixed by #584
Closed

Urinary Albumin Level (ACR) validation rules are wrong? #578

mbarton opened this issue Feb 9, 2025 · 2 comments · Fixed by #584
Assignees
@eatyourpeas
Labels
bug Something isn't working medium medium priority (no usage limits)

Comments

@mbarton
Copy link
Member

mbarton commented Feb 9, 2025

At the moment we say it must be 3-50 but in a real live dataset it ranges from 0-101. @eatyourpeas what's the expected range?
@mbarton mbarton added bug Something isn't working medium medium priority (no usage limits) labels Feb 9, 2025
@eatyourpeas
Copy link
Member

You are right.

220 mg/mmol defines nephrotic syndrome. I am not sure how high the assay goes to be honest. Maybe we should remove all validation around upper limits for this

@eatyourpeas eatyourpeas self-assigned this Feb 9, 2025
@eatyourpeas eatyourpeas linked a pull request Feb 10, 2025 that will close this issue
Albuminuria #584
Merged
@eatyourpeas
Copy link
Member

#584

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@eatyourpeas eatyourpeas

Labels
bug Something isn't working medium medium priority (no usage limits)
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Albuminuria rcpch/national-paediatric-diabetes-audit
2 participants
@mbarton @eatyourpeas