fix MLP

[jbrea]

Otherwise there is the following error

julia> MLJFlux.build(builder, rng, 3, 3)
ERROR: MethodError: Cannot `convert` an object of type 
  Flux.Dense{typeof(identity),Array{Float32{},2},Array{Float32{},1}} to an object of type 
  Flux.Dense{typeof(NNlib.relu),Array{Float32{},2},Array{Float32{},1}}
Closest candidates are:
  convert(::Type{T}, ::T) where T at essentials.jl:205
Stacktrace:
 [1] push!(a::Vector{Flux.Dense{typeof(NNlib.relu), Matrix{Float32}, Vector{Float32}}}, item::Flux.Dense{typeof(identity), Matrix{Float32}, Vector{Float32}})
   @ Base ./array.jl:932
 [2] build(mlp::MLJFlux.MLP{3}, rng::Random._GLOBAL_RNG, n_in::Int64, n_out::Int64)
   @ MLJFlux ~/.julia/dev/MLJFlux/src/builders.jl:93
 [3] top-level scope
   @ REPL[9]:1
 [4] top-level scope
   @ ~/.julia/packages/CUDA/YpW0k/src/initialization.jl:52

[ablaom]

Thanks indeed.

Oops, I have approved, but tests are not passing.

overlooked failing tests

[ablaom]

Mmm. Looks like julia 1.7 is behaving differently re RNGs. I thought we used StableRNGs everywhere except for dropout (needs this).

edit Correction. I see that StableRNGs are not used everywhere. Opening new issue: #195

[ablaom]

@jbrea Can you make a trivial commit to re-trigger buildkite? I think tests should pass now.

[codecov-commenter]

Codecov Report

Merging #194 (0838fa9) into dev (0b373f9) will increase coverage by 3.29%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              dev     #194      +/-   ##
==========================================
+ Coverage   88.97%   92.27%   +3.29%     
==========================================
  Files           9        9              
  Lines         245      246       +1     
==========================================
+ Hits          218      227       +9     
+ Misses         27       19       -8     

Impacted Files	Coverage Δ
src/builders.jl	100.00% <100.00%> (+36.36%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0b373f9...0838fa9. Read the comment docs.

[ablaom]

Thanks for this!