-
Notifications
You must be signed in to change notification settings - Fork 5
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
fix continue exposure #23
fix continue exposure #23
Conversation
I can review this weekend if you want and you think it’s ready |
Another validation exercise with I am pretty sure you will find the logic to the opened issue is fixed though. |
I don't have permissions to convert this to a draft PR but I'll be looking into this a bit deeper, would wait to review. |
Sorry, not quite following yet. What are the column headers on that screenshot? |
For this example: library(actxps)
library(dplyr)
census <- data.frame(pol_num = 1,
status = "Death",
issue_date = as.Date("2011-05-27"),
term_date = as.Date("2012-03-17"))
census <- rbind(census,data.frame(pol_num = 2,
status = "Death",
issue_date = as.Date("2011-05-27"),
term_date = as.Date("2012-09-17")))
exposed_cal <- census |>
expose(end_date = "2022-12-31", cal_expo = TRUE, target_status = "Death")
split <- expose_split(exposed_cal) I don't follow how this matches the documentation you quoted ("Application of the..."). Some questions I have looking at it:
Irrespective of what actexps is doing, let me think some more about calculating the year fraction on a policy versus calendar year basis. Besides the SOA paper, do you know any authoritative papers that get into this sort of thing? |
I'm not aware of any other authoritative resources. If some institution like SOA research institute could provide a dataset and maybe SQL implementation of expected functionality for experience analysis systems like that it would be quite valuable I feel. I raised a bug on Left cols are from Julia, right columns are from R. Nulls on right indicate left join didn't find matches. |
Closed in favor of #24 |
Just as a heads up, the example @alecloudenback shared looks odd because no value was passed to the Here's an adjusted version of the example.
An issue was created in actxps to catch situations where |
related to issue #22