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

Need more informative error #237

Closed ablaom closed 1 year ago

ablaom commented 1 year ago

https://github.com/FluxML/MLJFlux.jl/blob/4fa632de257cfde9dc0b01693b1f4270a4ecc405/src/mlj_model_interface.jl#L61

Throw-catch block here needs to throw the caught exception after the @error. At present users are hitting some uninformative errors.

@OkonSamuel