Kyle Daruwalla

Results 404 comments of 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...