remove abstract requirement of data_setter

[wd60622]

Description

I've removed the implementation for the CLV classes since the sample_posterior_predictive methods are not used in those models.

CC: @ColtAllen

Related Issue

• Closes #
• Related to #

Checklist

• Checked that the pre-commit linting/style checks pass. Feel free to comment pre-commit.ci autofix to auto-fix.
• Included tests that prove the fix is effective or that the new feature works
• Added necessary documentation (docstrings and/or example notebooks) using numpydoc format.
• If you are a pro: each commit corresponds to a relevant logical change

---

📚 Documentation preview 📚: https://pymc-marketing--1463.org.readthedocs.build/en/1463/

[wd60622]

Looking at how the customer choice model uses this, I am second guessing. For instance, if there is in sample posterior predictive then this wouldn't really make sense.

However, X is always required for the method at the moment. So maybe it makes sense...

What are people thoughts?

[ColtAllen]

I've removed the implementation for the CLV classes since the sample_posterior_predictive methods are not used in those models.

pymc.sample_posterior_predictive is being called for the plot_expected_purchases_ppc function. I haven't tested the ModelBuilder variant yet, but given how it's currently handling X data, CLV support might be possible.

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 92.72%. Comparing base (b0ba284) to head (2633134).

Files with missing lines	Patch %	Lines
pymc_marketing/model_builder.py	0.00%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1463      +/-   ##
==========================================
- Coverage   92.73%   92.72%   -0.02%     
==========================================
  Files          52       52              
  Lines        6114     6113       -1     
==========================================
- Hits         5670     5668       -2     
- Misses        444      445       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[wd60622]

pymc.sample_posterior_predictive is being called for the plot_expected_purchases_ppc function. I haven't tested the ModelBuilder variant yet, but given how it's currently handling X data, CLV support might be possible.

sample_posterior_predictive method and not the pymc function

[juanitorduz]

Thanks @wd60622 !