-
Notifications
You must be signed in to change notification settings - Fork 4
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
Review why it works #83
Comments
I'm confused about the last equation. Maybe I'm missing something obvious here. Shouldn't the last equation be: It looks like it's missing Otherwise, this all looks great to me. I think it's worth specifying the implicit assumption that the delay distribution doesn't change over time (see below). |
The use of integration by part is very neat. And so I'm guessing we can do the same with the derivation in the long paper, which might give slightly simpler notation and consistency? Starting from here (equations 12-14 in the long paper): If we assume that the forward distribution doesn't vary over time (such that Then, by using integration by parts, we get: which is same as equation 3.2 in the current vignette. And we can use the same treatment when we account for the truncation+censoring... (writing more later) |
Yes! Thats a typo (I think, I'll just double check myself) |
I spoke to @parksw3 a bit about this and if interested a review of the why it works sections would be great.
The text was updated successfully, but these errors were encountered: