sibylhe/mmm_stan

Definition formula of mc_df['mc_delta']

hiroki-kurasawa opened this issue · 3 comments

hiroki-kurasawa commented

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 commented

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

hiroki-kurasawa commented

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 commented

👌

  • 👍1