You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Here are some deviations between the left-hand side and our approximation on the right for various $n$:
$n=0$: 0 vs. nan
$n=1$: 0 vs. -1
$n=5$: 4.79 vs. 3.05
$n=10$: 15.10 vs. 13.03
$n=50$: 148.5 vs. 145.6
Before that change, we had been using $\ln(\Gamma(n+1))$, which agrees with $\ln(n!)$ by definition for integer $n$.
Whether this leads to a noticeable deviation of the assumed log-likelihood expression from a Poissonian one or even gives rise to pseudoexperiments with nan llh values depends on how fluctuations are produced and how large these are with respect to the expectations of course.
In any case, I think we should enable the use of an (actually Poissonian) log-likelihood metric, if need be with a new name so that existing analyses that have been using llh as it stands today can still be reproduced.