search

FluxML / FluxJS.jl

I heard you like compile times
Other
42 stars 8 forks source link

UndefVarError removed. #3

Closed ayush1999 closed 6 years ago

ayush1999 commented 6 years ago

Closes : #2 .

MikeInnes commented 6 years ago

I think this probably changed with Flux 0.5.0, is that the version you're on?

ayush1999 commented 6 years ago

Ah, I see. Sorry, I was on an older version. Although the change I made makes it compatible for 0.4.1 and newer versions also.

MikeInnes commented 6 years ago

I don't think we can support older flux versions here anyway, as we depend on things like the new convolution API,