Ethan Blackwood

@EricThomson thank you, will do!

@kushalkolar Thanks for the tips. It definitely makes sense to use `numpy.invert`, that's an oversight on their part. As for saving an intermediate `float32` memmap to feed into motion correction,...

Alright, after trying to implement a container that automatically inverts the data by subclassing np.memmap, I've realized this approach is a lot more limited and brittle than it initially seemed,...

I just realized that maybe we don't have to worry about memory-mapping sbx files? Since for motion correction, the `load` function only loads the data in smaller chunks at a...

I doubt this repo is even used anymore but this issue still exists and damn it's annoying

Hold off, it seems to break something (also getting this error in demo_seeded_CNMF.ipynb): ``` Traceback (most recent call last): File "/u/ethan/mesmerize-core/mesmerize_core/algorithms/cnmf.py", line 96, in run_algo cnm = cnm.fit(images) ^^^^^^^^^^^^^^^ File...

OK, this should fix it, but probably would be a good idea for someone else to check that notebook demo.

> caiman\source_extraction\cnmf\spatial.py:844: DeprecationWarning: Please use grey_dilation from the scipy.ndimage namespace, the scipy.ndimage.morphology namespace is deprecated. from scipy.ndimage.morphology import grey_dilation. (evaluate_components) I'm getting the same warning (repeated many many times) from...

I can do it no problem! I'll do a quick search for any other similar cases first.

So actually there is this comment in the same file: https://github.com/flatironinstitute/CaImAn/blob/bb55800806f0898592d79dcc705a0b53ccd01ec3/caiman/source_extraction/cnmf/spatial.py#L355-L359 which seems to imply that the local imports are needed for ipyparallel or multiprocessing workers. But I don't think...