I believe, but am not absolutely sure, this overloading of in is not needed in the latest versions of MLJBase.jl. It was needed by now-deprecated learning network apparatus. However, as it is likely needed by older versions of MLJBase.jl, it cannot be removed until the next breaking release of MLJModelInterface.jl.
ablaom
commented
9 months ago
The issue was detected by Aqua.jl running on a client package: https://github.com/JuliaAI/MLJScikitLearnInterface.jl/pull/64#issuecomment-1875712302
I believe, but am not absolutely sure, this overloading of
inis not needed in the latest versions of MLJBase.jl. It was needed by now-deprecated learning network apparatus. However, as it is likely needed by older versions of MLJBase.jl, it cannot be removed until the next breaking release of MLJModelInterface.jl.