forked from facebookexperimental/Robyn
-
Notifications
You must be signed in to change notification settings - Fork 1
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
Sync Resonance fork with latest Robyn #1
Open
DaveJacques
wants to merge
1,359
commits into
forward3d:main
Choose a base branch
from
facebookexperimental:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…modeling_new [modeling] add modeling implementations
remove the need for bool
…ion-interface visualizer interface
…ntal/Robyn into robynpy_release
…apper [feature] data mapper from R to Python
Robyn one pager end to end flow for tutorial 1
* added formatting with black-formatter * Added 4 new graphs to Pareto Optimizer + bug fixes * corrected notebooks * need to set cluster = true if you want bootstrap graph * Added Pareto utils call to src notebook --------- Co-authored-by: Dhaval Patel <funny@fb.com>
…_graph_twice Fix spend exposure graph twice
* refresh requirements * update requirements.txt * cleanup files * remove user paths
* debug modeling and pareto handoff * check feature parity for feature engineering * working modeling code * update allocator codes to include fixes for mroas and channel spends * cleared all notebook outputs * remove suppressed plt close * update prints to loggers * update print to loggers * fix instance logger files * fix logger
* refactored pareto_optimizer code * removed unused unit test --------- Co-authored-by: Dhaval Patel <funny@fb.com>
…al release (#1173) * revert ridge modeling to match convergence * tidy up loggers * clear notebooks, update gitignore * remove debug files
* refactored pareto_optimizer code * removed unused unit test * Added logging for Pareto module * cleaned up unused functions and unimplemented code * cluster loggers --------- Co-authored-by: Dhaval Patel <funny@fb.com>
* fix calculate decomp spend dist to use abs calc * clear notebook outs * update missing columns
* updated readme for python robyn * added package version of Robyn R used during python implementation --------- Co-authored-by: Dhaval Patel <funny@fb.com>
Moving tutorials into tests and organizing tests
* update visual fixes on notebook * clear notebook outputs * remove logger configuration on file levels * remove unused imports
* update colab notebooks * update with notebook content * publish with notebook content * update notebook code * update notebooks * update colab notebooks * update colab notebooks * update colab notebooks * update notebooks * update colab notebooks
Bumping up python release to 0.1.1
Introducing Robynpy - Robyn powered by Python.
Add Python version info to main readme.
…1132) * feat: enable exposure fitting - Exposures (Imp/GRP etc) will be prioritised over spend for parent model fitting - Deprecate function fit_spend_exposure, incl. Michaelis Menten. Nonlinear fitting between spend and exposure wasn't improving fitting significantly. Instead, future curve calibration feature will aim to improve curve identification. - Use cpe (cost per exposure) as ratio for spend to exposure translation. use cpe_window to scale the whole dataset in order to obtain the right spend scale for modeling period. - remove minpack.lm / nlsLM dependency - update exposure plot * feat: create robyn_calibrate function - Simulate cumulative R&F dataset with frequency bucket - add beta coef besides alpha and gamma to nevergrad hyperparameter to improve curve fit - plot with freq_bucket as well as onepager per trial - add df_curve_reach_freq as dummy dataset - create robyn_calibrate that consumes curve input and outputs hyperparameter ranges as input. - Rename previous internal robyn_calibrate function as lift_calibration - early stop convergence with while loop - update documentation * prototype: reach and frequency allocator This is the proof of concept of a R&F allocator that includes - Simulated R&F data - The R&F hill params are estimated using a multiplicative equation with Nevergrad - visualisation of surface - R&F allocator with nlopt - constrain validation * update: checks, input, transformation & website - simplify various check functions - adapt model.R, incl. reset run_transformations params to have clearer overview of params needed. simplify transformation.R by removing unnecessary checks - In model.R & pareto.R: remove decompSpendDist from both scripts to reduce memory leak. Use xDecompAgg subsets instead - In transformation.R & response.R: unify transformation namings in run_transformation and robyn_response - In response.R: remove exposure extrapolation because it's already done in robyn_input. Also add inflexion point to output. - In plots.R: fix onepager saturation plot issues - In pareto.R: rewrite run_dt_resp() as response_wrapper and align transformation logic & naming. - In pareto: Replace foreach response loop with lapply for simplicity. - In pareto.R: Simplify plot data generation process, esp for saturation curve plot, actual vs predicted plot & immediate vs carryover plot. - In pareto.R: Remove redundancy in xDecompVecCollect -> remove type rawMedia, rawSpend, predictedExposure, saturatedMedia & saturatedSpendReversed. Only keep adstockedMedia & decompMedia for response curve plotting. - add set_default_hyppar for easier testing - website update for all above changes
…te (#1090) Bumps [http-proxy-middleware](/~https://github.com/chimurai/http-proxy-middleware) from 2.0.6 to 2.0.7. - [Release notes](/~https://github.com/chimurai/http-proxy-middleware/releases) - [Changelog](/~https://github.com/chimurai/http-proxy-middleware/blob/v2.0.7/CHANGELOG.md) - [Commits](chimurai/http-proxy-middleware@v2.0.6...v2.0.7) --- updated-dependencies: - dependency-name: http-proxy-middleware dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [nanoid](/~https://github.com/ai/nanoid) from 3.3.6 to 3.3.8. - [Release notes](/~https://github.com/ai/nanoid/releases) - [Changelog](/~https://github.com/ai/nanoid/blob/main/CHANGELOG.md) - [Commits](ai/nanoid@3.3.6...3.3.8) --- updated-dependencies: - dependency-name: nanoid dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…e-pagers (#1194) * fix: prev versions' response curves and mean spends per channel on one-pagers
- Minor fix and tested on dummy data locally. Works as expected.
- minor fix and tested on local machine with different robyn_inputs specs, e.g. paid_media_spends and paid_media_vars are identical, or paid_media_spends only while paid_media_vars = NULL. All work as expected. - bump up version
…y InputCollect for BA bug (#1199) * fix: modeling window coverage considers all actual data dates (monthly data covers until end of month) * fix: adapt to rollingWindow too * fix: initial data message * fix: legacy paid_media_selected issue in BA * fix: replace paid_media_selected for paid_media_vars for response curves in OPs
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Project Robyn
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes # (issue)
Type of change
How Has This Been Tested?
Checklist: