Replies: 1 comment
-
Yikes thanks a lot for bringing this up! I don't have much bandwidth for this this week, but I could potentially next week, especially if you wanted to work on it together and maybe turn it into a contribution 😄 . I'm going to turn this into an issue so we can track it. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Hello,
This is mainly a documentation suggestion. My microscope saves Scanbox files, and I saw CaImAn has support for loading them, so I tried that but got an error because the loading functions are not up-to-date with newer versions of the file format (which has changed several times). The specific error was about inferring the number of channels (my
info
did not have'channels'
, but rather the newer'chan'
field).Looking into it a little more, I realized that the current function doesn't account for a number of features, including:
I took a few days to implement these features, and I was going to submit a pull request. However, towards the end I realized that the package sbxreader exists (which suite2p uses for loading). Besides being actively maintained, it's much better than what I wrote because it memory-maps the file - I realized when trying to test my function that it was not going to happen because my 40-minute recording was too large to fit into memory. It also has another function for loading all metadata, which is probably important to almost everyone who would need to load SBX data.
I could continue to tweak what I wrote to use memmap etc., but I think what makes the most sense is to just recommend Scanbox users to install this package and use it to get an np.memmap of their data, then load it into CaImAn using the movie constructor directly. To encourage this (and because it's out of date), it probably makes sense to remove the current SBX loading code.
Beta Was this translation helpful? Give feedback.
All reactions