search

FluxML / MLJFlux.jl

Wrapping deep learning models from the package Flux.jl for use in the MLJ.jl toolbox
http://fluxml.ai/MLJFlux.jl/
MIT License
145 stars 17 forks source link

added new builder for sklearn style mlp #188

Closed john-waczak closed 3 years ago

codecov-commenter commented 3 years ago

Codecov Report

Merging #188 (e3b3033) into dev (490a86b) will decrease coverage by 2.64%. The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev     #188      +/-   ##
==========================================
- Coverage   91.94%   89.30%   -2.65%     
==========================================
  Files           9        9              
  Lines         236      243       +7     
==========================================
  Hits          217      217              
- Misses         19       26       +7
Impacted Files Coverage Δ
src/builders.jl 66.66% <0.00%> (-33.34%) :arrow_down:

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 490a86b...e3b3033. Read the comment docs.

ablaom commented 3 years ago

@ToucheSir Thanks for chiming in here.

@john-waczak Would you mind submitting an updated PR?

john-waczak commented 3 years ago

@ablaom @ToucheSir

New PR submitted.