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

Re-add PACU model #204

Open
hqva opened this issue Jan 19, 2023 · 4 comments
Open

Re-add PACU model #204

hqva opened this issue Jan 19, 2023 · 4 comments
Assignees
Labels
discussion enhancement Update to and existin feature or request out-of-band
Milestone

Comments

@hqva
Copy link
Contributor

hqva commented Jan 19, 2023

Re-add PACU model

  • store in mlflow?
  • add column to front-end
@hqva hqva self-assigned this Jan 19, 2023
@hqva hqva added the enhancement Update to and existin feature or request label Jan 19, 2023
@docsteveharris docsteveharris added this to the HyMind milestone Feb 2, 2023
@hqva
Copy link
Contributor Author

hqva commented Feb 23, 2023

  • re-add medical history (LDA topic modelling vs simpler ICD10?)
  • re-train a model that produces good predictions
  • store and retrieve model from mlflow (I now know how to do this)
  • add column + colour-coding to rows based on whether patients should be flagged for further review

@docsteveharris
Copy link
Contributor

docsteveharris commented May 19, 2023

this is the same as https://github.com/HYLODE/HySys/issues/511
we should probably close the other and hold this one
for discussion next week as to when/whether we do this

@hqva
Copy link
Contributor Author

hqva commented May 19, 2023

Ah - if that issue is being closed:

I'd started this (not on FlowEHR or HySys) with a system that allows very quick change of model selection in this branch.

The relevant lines of code are here.

The predictions aren't great but it does work. branch hrishee/abacus which has essentially the same code (except with an xgboost model) is running on gae08:8343 now.

@harryjmoss
Copy link
Contributor

Model is deployed on the abacus deployment from @hqva - needs merging in

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion enhancement Update to and existin feature or request out-of-band
Projects
None yet
Development

When branches are created from issues, their pull requests are automatically linked.

4 participants