MMM data_df param renamed to data

[michaelraczycki]

contains renaming parameter from baseMMM from data_df to data requested in
issue #162 .

[juanitorduz]

Thanks @michaelraczycki ! Would you mind changing this as well in the example notebook https://pymc-marketing.readthedocs.io/en/stable/notebooks/mmm/mmm_example.html?

Actually, I wonder if it could be better to not just change the base class but all the arguments in the mmm module where data_df appears:

[michaelraczycki]

Sure, I can take care of that too.

[review-notebook-app]

Check out this pull request on 

See visual diffs & provide feedback on Jupyter Notebooks.

---

Powered by ReviewNB

[juanitorduz]

@ricardoV94 what do suggest doing in the meantime as the tests are not running (in view of #189)?

[ricardoV94]

Tests should pass after rebasing from main

[codecov]

Codecov Report

Merging #186 (f2f01bc) into main (d49b0b2) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #186   +/-   ##
=======================================
  Coverage   94.45%   94.45%           
=======================================
  Files          18       18           
  Lines         973      973           
=======================================
  Hits          919      919           
  Misses         54       54           

Impacted Files	Coverage Δ
pymc_marketing/mmm/base.py	95.00% <100.00%> (ø)
pymc_marketing/mmm/delayed_saturated_mmm.py	91.66% <100.00%> (ø)
pymc_marketing/mmm/preprocessing.py	100.00% <100.00%> (ø)
pymc_marketing/mmm/validating.py	100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.