StackOverflow on calling inverse VecCorrBijector with a multidimensional array

[mhauru]

using Bijectors: VecCorrBijector, inverse
inverse(VecCorrBijector())([[1.0]])

causes a stack overflow. It may be a non-sense call, but it should error in a more helpful, manageable way.

[penelopeysm]

Bijectors.jl/src/bijectors/corr.jl

Line 138 in d342371

function with_logabsdet_jacobian(::Inverse{VecCorrBijector}, y::AbstractVector{<:Real})

Broadening the type on y here would avoid the infinite recursion:

- function with_logabsdet_jacobian(::Inverse{VecCorrBijector}, y::AbstractVector{<:Real})
+ function with_logabsdet_jacobian(::Inverse{VecCorrBijector}, y)

julia> inverse(VecCorrBijector())([[1.0]])
ERROR: MethodError: no method matching AbstractFloat(::Type{Vector{Float64}})
The type `AbstractFloat` exists, but no method is defined for this combination of argument types when trying to construct it.

Closest candidates are:
  AbstractFloat(::Bool)
   @ Base float.jl:344
  AbstractFloat(::Int8)
   @ Base float.jl:345
  AbstractFloat(::UInt32)
   @ Base float.jl:352
  ...