-
Notifications
You must be signed in to change notification settings - Fork 6
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
Which models to keep? #15
Comments
@seabbs @parksw3 @kgostic do you think we have a consensus about which models to keep based on Friday's meeting? As I understand it, seems like we had mild agreement on deprecating any functions which were close to "pure" Perhaps I could start by making a PR to remove all of these, and in the meantime we can get clarity about which of the others to keep? |
I thought we had a roadmap which was to strip back to just the latent censoring and truncation model with feature issues to add support for retrospective (so no truncation), and the dynamic method (for edge cases/where only backwards distributions are available). If we want to keep any other models I think we should make adding them in as features and drop for now (so the current action can be to drop and add a issue review those models for later additions). |
ok, following the updated discussion, I think we're keeping this one: Line 171 in ff88f93
...and planning to drop everything else |
Sorry, I've been getting emails for all the issues, and I'm just getting to them after work, so I missed that I was tagged in this one... I also agree with Katie that we're only keeping one. |
(if the emails are annoying you can turn off github emails in the settings and just use github notifications) |
Thanks, that would be nice... I had to go through ~70 emails just now, which adds a huge mental block for me (more so than the actual task of reviewing issues)... |
We currently have a range of models that have been ported over from the research code base. @parksw3 we need to decide which of these to keep prior to any work being done on these.
I think my proposal would be keep just the latent truncation and censoring adjusted model and the dynamic backward -> forward models and with the view we will likely have a few more models flavours in the near future (as various improvements on those).
The text was updated successfully, but these errors were encountered: