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

Definition formula of mc_df['mc_delta'] #4

Closed
hiroki-kurasawa opened this issue Feb 5, 2021 · 3 comments
Closed

Definition formula of mc_df['mc_delta'] #4

hiroki-kurasawa opened this issue Feb 5, 2021 · 3 comments

Comments

@hiroki-kurasawa
Copy link

hiroki-kurasawa commented Feb 5, 2021

Hi Sibyl

thanks to this repository, I now have a deeper understanding of marketing mix modeling

I have a question about line 541 of mmm_stan.py.
https://github.com/sibylhe/mmm_stan/blob/main/mmm_stan.py#L541

in README, definition of delta is mc_pred - mc_ture, but delta is mc_true - mc_pred in line 541 . Is there any problem?

@sibylhe
Copy link
Owner

sibylhe commented Feb 8, 2021

Indeed! Fixed it. Thanks for pointing out this error!

@sibylhe sibylhe closed this as completed Feb 8, 2021
@hiroki-kurasawa
Copy link
Author

you should fix it as mc_df['mc_delta'] = mc_df['mc_pred'] - mc_df['mc_true'] instead of mc_df['mc_delta'] = ['mc_pred'] - mc_df['mc_true']

https://github.com/sibylhe/mmm_stan/blob/main/mmm_stan.py#L541

@sibylhe
Copy link
Owner

sibylhe commented Feb 9, 2021

👌

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

No branches or pull requests

2 participants