Implement ModifiedBetaGeoNBD and ModifiedBetaGeoNBDRV

[PabloRoque]

Description

This PR is equivalent to #1317, now tackling ModifiedBetaGeoNBD and ModifiedBetaGeoNBDRV.

• Adds ModifiedBetaGeoNBD distribution
• Adds ModifiedBetaGeoNBDRV
• Introduces recency_frequency sampling from ModifiedBetaGeoNBD in ModifiedBetaGeoModel.distribution_new_customer

Related Issue

• Closes #
• Related to CLV API Standardization #527

Checklist

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

Modules affected

• MMM
• CLV
• Customer Choice

Type of change

• New feature / enhancement
• Bug fix
• Documentation
• Maintenance
• Other (please specify):

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.65%. Comparing base (75ee409) to head (239545b).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1375      +/-   ##
==========================================
+ Coverage   94.59%   94.65%   +0.05%     
==========================================
  Files          48       48              
  Lines        5202     5257      +55     
==========================================
+ Hits         4921     4976      +55     
  Misses        281      281              

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

[juanitorduz]

Thank you @PabloRoque ! It looks good to me, but I would ask @ColtAllen for a detailed review before merging 🙏

[PabloRoque]

Thank you @PabloRoque ! It looks good to me, but I would ask @ColtAllen for a detailed review before merging 🙏

Thanks for having a look.

Let's wait for @ColtAllen review. I am sure he'll come up with some nice suggestions.

[ColtAllen]

There's a lot of overlapping work here with #1390. Let's go ahead and merge this so the other PR can just focus on covariates. I'll double-check the docs before the next release.

My mistake - the sim_data function looks good!