Skip to content

Conversation

@brookslogan
Copy link
Contributor

Checklist

Please:

  • Make sure this PR is against "dev", not "main" (unless this is a release
    PR).
  • [-] Request a review from one of the current main reviewers:
    brookslogan, nmdefries.
  • [-] Makes sure to bump the version number in DESCRIPTION. Always increment
    the patch version number (the third number), unless you are making a
    release PR from dev to main, in which case increment the minor version
    number (the second number).
  • [-] Describe changes made in NEWS.md, making sure breaking changes
    (backwards-incompatible changes to the documented interface) are noted.
    Collect the changes under the next release number (e.g. if you are on
    1.7.2, then write your changes under the 1.8 heading).
  • See DEVELOPMENT.md for more information on the development
    process.

Change explanations for reviewer

Release 0.10. Version has already been bumped on dev.

Magic GitHub syntax to mark associated Issue(s) as resolved when this is merged into the default branch

  • Resolves #{issue number}

dshemetov and others added 30 commits October 24, 2024 10:45
docs(complete.epi_df): fix backticks, tweak 1-line summary
doc: update from Rachel's comments
- BREAKING CHANGE: adjust default output column naming scheme, disallow
  overwriting columns.
- Show naming options, including with multi-column selections when we have
  accommodating example data sets
- Select away the pre-existing 7d aggregations in the example data set
- Ungroup output
We may later need it to make default args, in which case we should probably
export to be less "magical".
Co-authored-by: nmdefries <42820733+nmdefries@users.noreply.github.com>
@brookslogan brookslogan marked this pull request as draft December 10, 2024 06:45
@brookslogan
Copy link
Contributor Author

brookslogan commented Dec 10, 2024

Improve `epi_slide_{sum,mean}` unsupported-arg messages.
@brookslogan brookslogan marked this pull request as ready for review December 10, 2024 06:59
@brookslogan brookslogan merged commit 0e09c4a into main Dec 10, 2024
8 checks passed
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.

4 participants