FluxML / FluxJS.jl

I heard you like compile times

Other

42 stars 8 forks source link

fix #17 #18

Closed Roboneet closed 6 years ago

Roboneet commented 6 years ago

Code from https://github.com/FluxML/Flux.jl/commit/73a0be3e04e37fd1061cd964100c2d1ee589424b

17 seems to be resolved, but #2 has come up again. Tracker is not exported in the current version of Flux (https://github.com/FluxML/Flux.jl/commit/e186b958ddd01c04512bc6d3bbf7556c2550ba5c). https://github.com/FluxML/FluxJS.jl/pull/3 would solve it

MikeInnes commented 6 years ago

Thanks! I just re-exported tracker, so that should be fixed as well now.