Break ModelBuilder into smaller classes

[williambdean]

Description

Splitting the original ModelBuilder into base class which then supports ModelBuilder and RegressionModelBuilder

Work in progress. Likely very broken at the moment

Related Issue

• Closes Consolidate CLV load with ModelBuilder classmethod #1380
• Related to Option to exclude fit data when saving models #1356

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--1467.org.readthedocs.build/en/1467/

[codecov]

Codecov Report

❌ Patch coverage is 55.55556% with 60 lines in your changes missing coverage. Please review.
✅ Project coverage is 20.77%. Comparing base (488ac3c) to head (6416c01).

Files with missing lines	Patch %	Lines
pymc_marketing/model_builder.py	54.68%	58 Missing ⚠️
pymc_marketing/clv/models/basic.py	33.33%	2 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (488ac3c) and HEAD (6416c01). Click for more details.

HEAD has 22 uploads less than BASE

Flag	BASE (488ac3c)	HEAD (6416c01)
	23	1

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #1467       +/-   ##
===========================================
- Coverage   92.28%   20.77%   -71.52%     
===========================================
  Files          64       64               
  Lines        7429     7448       +19     
===========================================
- Hits         6856     1547     -5309     
- Misses        573     5901     +5328     

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[ColtAllen]

#1356 should be considered for the load method replacing this.

[ColtAllen]

Resolved merge conflicts to resume work on this because it overlaps with #1377, #1356, and #1380.

@williambdean Do you anticipate the Bass module inheriting from ModelBuilder in the future?

[williambdean]

Resolved merge conflicts to resume work on this because it overlaps with #1377, #1356, and #1380.

Thanks!

@williambdean Do you anticipate the Bass module inheriting from ModelBuilder in the future?

Maybe

[ColtAllen]

This PR is superseded by #1870, but if it makes more sense to merge into this branch rather than main, let me know.

[williambdean]

Feel free to close this one out @ColtAllen