Associations.jl icon indicating copy to clipboard operation
Associations.jl copied to clipboard

Unnecessary double call to `counts` for `association(::JointProbabilities{<:MIShannon}, x, y)`

Open kahaaga opened this issue 6 months ago • 0 comments

In the file methods/information/definitions/mutual_informations/MIShannon.jl:

The cts = counts(est.discretization, x, y) line is completely unnecessary. The counts are automatically computed when calling probabilities on the line below. This extra line essentially doubles computation time.

kahaaga avatar Aug 02 '24 09:08 kahaaga