-
Notifications
You must be signed in to change notification settings - Fork 5
Consider adding fetch functions returning epiprocess
objects
#91
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
Comments
fetch_edf()
returning an epi_df
?fetch_edf()
/fetch_epi_df()
returning an epi_df
?
fetch_edf()
/fetch_epi_df()
returning an epi_df
?fetch_edf()
/fetch_epi_df()
?
I'm thinking we should circle back to this after both #72 is addressed and the v1.0.0 release. A few thoughts on how this might work:
|
fetch_edf()
/fetch_epi_df()
?epiprocess
objects
Just realized we might also want to get |
Adding |
|
It might actually require more thinking about how to handle |
Two separate but related points:
Advantages:
|
To streamline analyses using our new package ecosystem, it might help to be able to get
epi_df
s directly out ofepidatr
. This could be opt-in withSuggests: epiprocess
and requiring it insidefetch_edf
. However, at minimum, this would involve renaming and setting key columns on a per-endpoint basis, and we might also consider pivoting to a wider format. Thus, it's a nontrivial amount of work. (But also hopefully a nontrivial amount of savings when actually being used.)Might conflict with the goal of #72.
The text was updated successfully, but these errors were encountered: