wheeheee

Results 62 comments of wheeheee

Interesting, but not sure if it's gonna be useful for most of the current user base?

Hm. Well, typically `n_channels` is quite small compared to `n_samples`. We could write a heuristic to detect if it's the other way round and emit a warning if so. But...

I still think pretty much all cases will be caught by the heuristic, but I guess the keyword argument method is safer and isn't too much work. However, we probably...

That also sounds reasonable. But wait, if we instead let the default be say `nothing`, then let it error by default, we could also provide instructions on how to fix...

Yeah, I was thinking something like `f(s; argument_order)` instead of `f(s; argument_order=nothing)` at first... Anyway, we might be worrying too much, because there seems to be only one package ([NeuroAnalyzer.jl](https://codeberg.org/AdamWysokinski/NeuroAnalyzer.jl))...

Oh, and NeuroAnalysis.jl. The package names looked similar...

Yeah, that's true for most of the configs. Maybe a warning is warranted but OTOH it should be obvious because it is stated that buffers are pre-allocated.

Then again, that isn't really implicit in the common understanding of config. I guess this should be done...

You could just rename the lowercase files to `*_core.jl`, `*_main.jl`, or even just `_*.jl`. That solves the problem with case-insensitivity on Windows.

A lot of these are personal preferences, some communities like SciML have style guides, and there are recommendations on Julia's discourse if you search for them