schnitzer-lab / EXTRACT-public

EXTRACT is a tractable and robust automated cell extraction tool for calcium imaging, which extracts the activities of cells as time series from both one-photon and two-photon calcium imaging movies.
MIT License
62 stars 16 forks source link

Easier parsing of `compact_output` for cell refinement #35

Open bahanonu opened 1 year ago

bahanonu commented 1 year ago

@fatihdinc Likely already mentioned this previously, but using the output of compact_output=false might be improved to make it easier for users to determine which parameters to change in their movies, especially as it pertains to the Cell refinement module to reduce false negatives/positives.

It would be good to have either:

fatihdinc commented 1 year ago

Thanks Biafra. For now, I use the code "plot_hyperparameter_curves(output)" for checking these metrics. Even if compact_output = True, these metrics will include the metrics corresponding to "bad" cells. I am working on a better GUI for this, thank you for the heads up!

Compact_output is set to false by default, as well as the sparse_array option, to prevent memory issues and performance degradation in large movies. Like you suggested, these do not matter much for miniscope etc. movies.

jbscherrer commented 1 year ago

@fatihdinc the plot_hyperparameter_curves function looks like it would be pretty helpful in optimizing cellchecking hyperparameters, however it looks like the cutoffs are hardcoded in and don't always correspond to the current default values. Am I reading that right?

fatihdinc commented 1 year ago

That is indeed correct! I will hopefully focus on user friendliness soon, hopefully supplying these functions with video lectures and tutorials.