Exclude prior suffix from CLV model config #1428
Labels
CLV
good second issue
Bit more involved but still doable for newcomers
maintenance
major
API breaking changes
model config
request discussion
The MMM and customer choice models don't have this suffix. All CLV models have this convention. Here is an example:
pymc-marketing/pymc_marketing/clv/models/beta_geo.py
Lines 169 to 172 in d9f0c51
I think the "prior" is implied in the context of the model so I find it a bit redundant. Then the variable drops the suffix in the model.
What are your thoughts @ColtAllen?
The text was updated successfully, but these errors were encountered: