remove abstract method 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 94.61%. Comparing base (0750eba) to head (edb4ae5).
Report is 1 commits behind head on main.

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   94.70%   94.61%   -0.10%     
==========================================
  Files          50       50              
  Lines        5458     5457       -1     
==========================================
- Hits         5169     5163       -6     
- Misses        289      294       +5     

☔ 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