Skip to content

Add a transform_forecasts() function#278

Merged
nikosbosse merged 18 commits intomasterfrom
add_scale_fct_no_trunc
Apr 11, 2023
Merged

Add a transform_forecasts() function#278
nikosbosse merged 18 commits intomasterfrom
add_scale_fct_no_trunc

Conversation

@nikosbosse
Copy link
Copy Markdown
Collaborator

This proposal is very similar to #271, with the only difference that the log_shift() function has no negative_to_zero argument. This seemed somewhat controversial, so this PR introduces the main transform_forecasts() function and we can think about adding the function argument at a later time.

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 10, 2023

Codecov Report

Merging #278 (dcffbf4) into master (66487aa) will decrease coverage by 0.27%.
The diff coverage is 81.57%.

❗ Current head dcffbf4 differs from pull request most recent head d2b3132. Consider uploading reports for the commit d2b3132 to get more accurate results

@@            Coverage Diff             @@
##           master     #278      +/-   ##
==========================================
- Coverage   90.82%   90.55%   -0.27%     
==========================================
  Files          21       22       +1     
  Lines        1286     1324      +38     
==========================================
+ Hits         1168     1199      +31     
- Misses        118      125       +7     
Impacted Files Coverage Δ
R/convenience-functions.R 81.57% <81.57%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@nikosbosse nikosbosse merged commit 2066e47 into master Apr 11, 2023
@nikosbosse nikosbosse deleted the add_scale_fct_no_trunc branch April 11, 2023 06:47
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.

1 participant