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

Readmission of previously well babies #9

Open
tomjemmett opened this issue Nov 1, 2024 · 6 comments · Fixed by The-Strategy-Unit/nhp_project_information#163
Open
Assignees
Labels
question Further information is requested

Comments

@tomjemmett
Copy link
Member

There is a subtle difference between the old logic used when we extracted from our Sql server and what we are using here.

Previously, we join the nhp inpatients view to itself. We now join the nhp inpatients table (which only includes a subset of providers) to the base inpatients table (which includes all providers).

The subtle difference is in the Sql version, the inpatients view is filtered to remove well babies. So, our new logic is including more "readmissions" because we have cases where a baby is readmitted within 28 days of birth, but we previously didn't count this as a readmission because we were filtering out the well baby rows.

I think that this is an oversite, and the new logic is correct. This issue is being created purely to note this difference and allow anyone to have a differing opinion.

@tomjemmett tomjemmett self-assigned this Nov 1, 2024
@tomjemmett tomjemmett added the question Further information is requested label Nov 1, 2024
@StatsRhian
Copy link
Member

  • Needs to be checked by Jake
  • Then added to data log on project information

@tomjemmett
Copy link
Member Author

For v3.0.0 matched the logic in v2.2.0. Keeping issue open so we can review in the next sprint

@yiwen-h
Copy link
Member

yiwen-h commented Nov 25, 2024

  • Discussion for analytical steering group
  • Add clarification to project information
  • Flag with Alex - possibility to include with paeds mitigators? - Analyse what conditions these previously well babies are being readmitted with

@yiwen-h yiwen-h assigned yiwen-h and unassigned tomjemmett Nov 25, 2024
@yiwen-h
Copy link
Member

yiwen-h commented Dec 13, 2024

@tomjemmett would like to do more work on this before it goes to analytical steering group

@yiwen-h
Copy link
Member

yiwen-h commented Dec 16, 2024

@yiwen-h
Copy link
Member

yiwen-h commented Dec 23, 2024

This is no longer a data science issue. Have scheduled meeting with @DandyOoh ; if not suitable for the paeds mitigators, will pass to Mitigator Review process.

@yiwen-h yiwen-h closed this as completed Dec 23, 2024
@yiwen-h yiwen-h reopened this Jan 6, 2025
@yiwen-h yiwen-h assigned DandyOoh and unassigned yiwen-h Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants