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

Add vignette on slowly changing dimensions #53

Merged
merged 5 commits into from
Oct 16, 2023

Conversation

marcusmunch
Copy link
Collaborator

@marcusmunch marcusmunch commented Oct 5, 2023

Intent

This PR introduces a vignette describing the concept of a slowly changing
dimension using a couple of examples (forecasts changing daily compared to
civil registry addresses changing slowly).

Approach

The PR adds a vignette which may be viewed with vignette("slowly-changing-dimension")

Known issues

The name is a bit long.

Checklist

  • The PR passes all local unit tests
  • I have documented any new features introduced
  • If the PR adds a new feature, please add an entry in NEWS.md
  • A reviewer is assigned to this PR

@marcusmunch marcusmunch added the documentation Improvements or additions to documentation label Oct 5, 2023
@marcusmunch marcusmunch self-assigned this Oct 5, 2023
vignettes/slowly-changing-dimension.Rmd Outdated Show resolved Hide resolved

# Current forecasts
forecasts_full %>%
slice_max(ForecastDate, n = 1) %>%
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

n = 1 is used by default, so you don't need to specify it if you don't want to

vignettes/slowly-changing-dimension.Rmd Show resolved Hide resolved
@marcusmunch marcusmunch merged commit 2694eba into ssi-dk:main Oct 16, 2023
6 checks passed
@marcusmunch marcusmunch deleted the scd-vignette branch October 16, 2023 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants