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

Consider adding join/merge between epi_archive and data frame #148

Open
brookslogan opened this issue Jul 18, 2022 · 1 comment
Open

Consider adding join/merge between epi_archive and data frame #148

brookslogan opened this issue Jul 18, 2022 · 1 comment
Labels
enhancement New feature or request P2 low priority

Comments

@brookslogan
Copy link
Contributor

Add a join/merge function to combine an epi_archive with non-versioned data in a data frame, e.g., to:

  • add unversioned population data
  • add unversioned epi data from an epi_df
  • potentially, add unversioned data without a geo_value from a data frame

Naming might need thought, since R6 $merge is already taken with S3 epix_merge.

@brookslogan brookslogan added enhancement New feature or request P2 low priority labels Jul 18, 2022
@brookslogan
Copy link
Contributor Author

The existing $merge allows for this, but does not document it, and probably leaves the archive DT with an invalid key. The new $merge in #101 does not allow for this (but a user could hack it by merging with an epi_archive considering all data observed in 1000BCE and nonclobberable).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request P2 low priority
Projects
None yet
Development

No branches or pull requests

1 participant