Closed Datseris closed 1 year ago
The TODOs comments are old and this should now be fixed.
Shrinkage
estimator is already tested with SpatialDispersion
. probs_and_outs_from_histogram
to actually estimate the probabilities, the only possible point to cause errors is at the level of probabilities_and_outcomes(est::Shrinkage, x)
and allprobabilities_and_outcomes(est::Shrinkage, x)
. From what I can see, these methods dispatch correctly now.
The
Shrinkage
implementation has the following TODOs that we should fix at some point:probabilities
andallprobabilities
.SymbolicDispersion
estimator(both should have coresponding tests)