-
Notifications
You must be signed in to change notification settings - Fork 23
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
Why/how are events dropped which occur at the same time? #56
Comments
This package takes the same approach as |
Got it. Thanks for clarifying. Given that using unique times is the norm, I will close this issue. |
Not related to the issue as stated but it's interesting that in your figure Julia has a lower estimate of the hazard than R. I'm not sure why that is. |
Yeah. That was part of my concern. I don't know how the algorithm works, but my naive hypothesis is that increasing the number of observations occurring at the same time, leads to an increase in the cumulative hazard. This is consistent with the observation that Julia and R agree when the duplicates are removed. I think it could be further tested by increasing the duplicates. |
AFAICT, this appears to be because the mice function |
Hello,
I noticed that Julia and R differ on an example from the MICE documentation. Initially, I thought this was related to #50, but upon closer inspection, I noticed that Julia removes duplicate times (R and Julia agree when duplicates are removed). It might be a good idea to explain this in the documentation, or perhaps not remove duplicates (return a warning instead?). I'm not sure what the best approach is or whether there is a best approach, but I think documenting the difference from R would be helpful to users.
R Code
require(MASS)
Julia Code
Version Info
The text was updated successfully, but these errors were encountered: