CaImAn icon indicating copy to clipboard operation
CaImAn copied to clipboard

Do you need to downsample the video for calculating the correlation image, if you downsampled using ssub,tsub ??

Open bashsyr opened this issue 10 months ago • 1 comments

Your setup:

  1. Operating System (Linux, MacOS, Windows): Windows
  2. Hardware type (x86, ARM..) and RAM: x86 RAM:64GB
  3. Python Version (e.g. 3.9): 3.10.8
  4. Caiman version (e.g. 1.9.12):1.9.15
  5. Which demo exhibits the problem (if applicable): pipeline_cnmfe
  6. How you installed Caiman (pure conda, conda + compile, colab, ..): usual mamba setup in Windows liek the README
  7. Details:

Hello, I basically wrote the question in the title. In #815 it was mentioned that when you downsample min_pnr is automatically adjusted here min_corr is not. But that was in 2020 and I couldn"t find any up to date answers So my question now is: do I need to downsample the video manually for calculating the cn_filter,pnr in here or is it done automatically ? Thank you so much for your answer

bashsyr avatar Aug 22 '23 15:08 bashsyr

The code hasn't been changed for these calculations since that issue was opened, so whatever @j-friedrich said then still holds. I will need to go through and look at this and get back to you with a clear answer.

At some point it would be good to write up a table or list enumerating exactly what you have to keep track of when you subsample (e.g., gSig, gSiz, etc). That code you linked to is pretty hard to parse (for this human), and we should probably try to make it a bit more explicit and readable as a code sprint/refactor goal (e.g., it would be good to tweak the param outside of the function call). ONe of the early to-do list items is refactoring initialization functions so I will work on this more but it is a few months away.

EricThomson avatar Aug 22 '23 15:08 EricThomson