Fix erroneous change of constructor signature. (#427)

* Fix erroneous change of constructor signature.

The `where V` must be *inside* the outer brackets, because the element type of the outcome vectors may be different (when categorical data are used - we can mix integers, strings, etc).

* Update changelog and increment version

* fix version/changelog and add test

Project.toml

@@ -2,7 +2,7 @@ name = "ComplexityMeasures"
 uuid = "ab4b797d-85ee-42ba-b621-05d793b346a2"
 authors = "Kristian Agasøster Haaga <[email protected]>, George Datseries <[email protected]>"
 repo = "https://github.com/juliadynamics/ComplexityMeasures.jl.git"
-version = "3.7.0"
+version = "3.7.1"
 
 [deps]
 Combinatorics = "861a8166-3701-5b0c-9a16-15d98fcdc6aa"

src/core/probabilities.jl

@@ -57,7 +57,7 @@ struct Probabilities{T, N, S} <: AbstractArray{T, N}
     dimlabels::NTuple{N, S}
 
     function Probabilities(x::AbstractArray{T, N},
-            outcomes::Tuple{Vararg{V, N}} where V,
+            outcomes::Tuple{Vararg{V, N} where V},
             dimlabels::NTuple{N, S};
             normed::Bool = false) where {T, N, S}
         if !normed # `normed` is an internal argument that skips checking the sum.

test/probabilities/probabilities.jl

@@ -11,6 +11,11 @@ outs = collect(1:10)
 @test Probabilities(rand(rng, 10), (outs,)) isa Probabilities
 @test Probabilities(rand(rng, 10), (outs,), (:x1, )) isa Probabilities
 
+# outcome vectors for different eltypes
+outs1 = ["a", "b"]; outs2 = [1, 2];
+cts = rand(1:10, 2, 2);
+@test Probabilities(cts, (outs1, outs2)) isa Probabilities
+
 # ----------------------------------------------------------------
 # Base extensions 
 # ----------------------------------------------------------------