bachmannpatrick / CLVTools

R-Package for estimating CLV
54 stars 14 forks source link

Dyncov plot in some cases 1 period shifted from actuals #78

Open pschil opened 4 years ago

pschil commented 4 years ago

The actual and expected repeat transactions are shifted by a period because they currently are (wrongfully) defined differently: Repeat transactions are currently implemented as "forward-looking", ie the number of transactions in the period that starts at the plotting point whereas expected transactions are implemented as "backward-looking", ie expected transactions in the period that end at the plotting point.

The BTYD package implements this backward-looking which can be seen from the plot which at the estimation start has a hard-coded zero for the expectation (see BTYD::dc.CumulativeToIncremental). For consistency with the plots from BTYD, the plot in CLVTools is also implemented as backward-looking.

This requires to implement:

pschil commented 4 years ago

Still existing issues

Kink Because of the dyncov model, the plotting points are currently set to be on the covariate dates. This often causes the second plotting point to be spaced less than 1 period from the first plotting point (estimation start) what again implies that the expectation at the second plotting point is calculated only as a partial period. This partial period leads to a "kink" in the plot at the beginning.

Shifted Periods For some cases, the exepectation values are still shifted by what seems exactly 1 period from the actual transactions.

pschil commented 4 years ago

Postponed to next release. The kink is not so much of an issue and the shifted periods could not be found, yet.