Kyle Daruwalla
Kyle Daruwalla
Makes a lot of sense to me. Maybe we should rename `ObsView` to `LazyView` to indicate that it is both a view (subset) of the observations as well as being...
I believe FFCV has the notion of a [traversal order](https://docs.ffcv.io/api/loader.html) which we might want to look into. Apparently the quasi-random variant increases performance too, so there may be a third...
I guess the main issue here is taking on CUDA as a dep. The fix is easy.
Yes, this would be a nice addition. You can probably adapt [the MLDataPattern.jl code](https://github.com/JuliaML/MLDataPattern.jl/blob/master/src/stratifiedobs.jl) as a starting point. I would try and start with the simplest implementation first though, since...
Is the plan to make changes here directly instead of "fixing" the existing packages? What's the timeline here? I would say we at least need a couple of cycles with...
> So the plan could be to finish the port in this repo where I'm gradually reviewing and modernizing the whole codebase, then push the whole thing into https://github.com/JuliaML/MLDataUtils.jl, tag...
@johnnychen94 do you have any thoughts on this plan?
Seems like a good idea to me. First we can deprecate `normalise` for `standardise`, then release real normalization in a breaking release. We'll need PRs to downstream libraries like Flux...
The repo https://github.com/AStupidBear/SpikingNeuralNetworks.jl has implementations of these models that might be useful reference.
I'll take a look at this sometime this week. Did you use a released version of the package or the master branch? This repo is mid-refactor so I haven't done...