Skip to content

Conversation

@brookslogan
Copy link
Contributor

No description provided.

kenmawer and others added 30 commits August 9, 2022 09:51
- `missing(..1)` is a bit unclear, and can fail given, e.g., `,,,,10,,` as the
  dots
- `missing(...)` and `length(...)` are not available until pretty recent R
  versions
Merge branch 'main' of https://github.com/cmu-delphi/epiprocess into km-slide-n-replace2.1

# Conflicts:
#	man/epi_slide.Rd
nmdefries and others added 26 commits June 6, 2023 11:58
so developers can use (untracked) renv's at the repo level without as many git
and CHECK complaints.
…imevalue-without-env

Pass `ref_time_value` to `epix_slide` for functions and formulas
Set DELPHI_EPIDATA_KEY, from a GitHub Secret
and add some unrelated comments regarding time value counting details.

Since we're not potentially dealing with R6 objects, we can just us
`as_data_mask(.x)`. We can also flatten the data mask by installing the `.x`,
`.group_key` and `.ref_time_value` in the same way as that pronouns are
installed (except keeping their original classes, not converting to pronouns),
rather than using another level in the environment chain for the data mask.
Allow `epi_slide` to access `ref_time_value`
Pass ref time value and group key to epix_slide for tidy computations
…imevalue-without-env

Sync `dev` with `.ref_time_value` and semi-pronoun changes
@brookslogan brookslogan requested a review from dajmcdon as a code owner June 17, 2023 00:37
@brookslogan brookslogan merged commit 2f873ca into main Jun 20, 2023
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

Successfully merging this pull request may close these issues.

6 participants