clamp gives TypeError from julia 1.11

[visr]

using SparseConnectivityTracer
detector = TracerSparsityDetector()
jacobian_sparsity(x -> clamp!(x, 0.1, 0.9), rand(3), detector)

On Julia 1.10.5 this gives:

3×3 SparseArrays.SparseMatrixCSC{Bool, Int64} with 3 stored entries:
 1  ⋅  ⋅
 ⋅  1  ⋅
 ⋅  ⋅  1

But on Julia 1.11 and nightly it gives:

ERROR: TypeError: non-boolean (SparseConnectivityTracer.GradientTracer{SparseConnectivityTracer.IndexSetGradientPattern{Int64, BitSet}}) used in boolean context
Stacktrace:
 [1] clamp(x::SparseConnectivityTracer.GradientTracer{…}, lo::Float64, hi::Float64)
   @ Base .\intfuncs.jl:1266
 [2] clamp!
   @ .\intfuncs.jl:1322 [inlined]
 [3] (::var"#237#238")(x::Vector{SparseConnectivityTracer.GradientTracer{…}})
   @ Main .\REPL[33]:1
 [4] trace_function(::Type{SparseConnectivityTracer.GradientTracer{…}}, f::var"#237#238", x::Vector{Float64})
   @ SparseConnectivityTracer C:\Users\visser_mn\.julia\dev\SparseConnectivityTracer\src\trace_functions.jl:47
 [5] _jacobian_sparsity(f::Function, x::Vector{Float64}, ::Type{SparseConnectivityTracer.GradientTracer{…}})
   @ SparseConnectivityTracer C:\Users\visser_mn\.julia\dev\SparseConnectivityTracer\src\trace_functions.jl:73
 [6] jacobian_sparsity(f::Function, x::Vector{…}, ::TracerSparsityDetector{…})
   @ SparseConnectivityTracer C:\Users\visser_mn\.julia\dev\SparseConnectivityTracer\src\adtypes_interface.jl:56
 [7] top-level scope
   @ REPL[33]:1

I suspect the reason is moving clamp from isless on 1.10 to the ternary operator on 1.11.
This was changed in JuliaLang/julia#54038.

[adrhill]

Thanks, this is definitely a function SCT should support.
So far, we haven't covered many three-argument functions, but this one should be doable.

[adrhill]

I tagged a v0.6.8 release which includes support for clamp and clamp!.

[visr]

Thanks a lot!