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

docs: Preparing CRAN version #1000

Merged
merged 2 commits into from
Jun 19, 2024
Merged

docs: Preparing CRAN version #1000

merged 2 commits into from
Jun 19, 2024

Conversation

laresbernardo
Copy link
Collaborator

Preparing CRAN version

dependabot bot and others added 2 commits June 17, 2024 04:01
… upper constraints fix when higher than mean (#974)

* fix: refresh hyps check #960 + use data available in json

* fix: update based on gz's comments

* fix: fixed penalties and other fixed hyps on refreshing models

* fix: refresh plot when chain is broken + feat: new bounds_freedom parameter to overwrite default calculation

* fix: import and store original model when not in original plot_dir

* recode: applied styler::tidyverse_style() to clean code for CRAN

* fix: paid_media_total calc

* fix: print ExportedModel only when available

* fix: deal with negative trend

- negative trend is not interpretable for MMM
- force negative coef when trend is negative to get positive decomp

* fix: upper constraint issue on BA for target_efficiency and weibull adstock

feat: instead of Inf, use channel_constr_up, which by default is 10 for target_efficiency

* fix: reverse wrong bounds update in refresh_hyps

The refactoring of initBounds & listOutputPrev in refresh_hyps was wrong in 774c18d

* recode: apply styler::tidyverse_style()

---------

Co-authored-by: gufengzhou <gufengzhou@gmail.com>
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jun 19, 2024
@laresbernardo laresbernardo merged commit ad3f2eb into cran_3.11.0 Jun 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants