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

[discussion] epix_as_of(as_of) default setting #608

Open
dshemetov opened this issue Feb 21, 2025 · 1 comment
Open

[discussion] epix_as_of(as_of) default setting #608

dshemetov opened this issue Feb 21, 2025 · 1 comment

Comments

@dshemetov
Copy link
Contributor

I end up writing this a lot to get the latest snapshot from an archive: epix_as_of(min(Sys.Date(), nhsn_archive_data$versions_end)). Let's discuss the pros and cons of having that be default.

TODO

@brookslogan
Copy link
Contributor

I often write %>% epix_as_of(.$versions_end). That's also a bit annoying, especially if trying to use base R pipe.

  • Is there any case where we would have versions /past/ Sys.Date() and want to use Sys.Date() instead? Could the above be used instead?
  • I've been thinking maybe we could just have another function epix_latest() or something for this operation. Do you think it's worth a separate name like this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants