spikeinterface icon indicating copy to clipboard operation
spikeinterface copied to clipboard

Analyzer set recording

Open alejoe91 opened this issue 1 year ago • 3 comments

This is useful for example if the analyzer was set up and ran with a tmp cached recording, but later on one wants to set a lazy recording to perform additional operations and computations

alejoe91 avatar Apr 30 '24 12:04 alejoe91

I really like this idea because we often do the analysis and then move everything over to our server for long-term storage which means the recording-analyzer link gets broken. My only concern (and this is now from experience helping others) is that people forget what goes with what. So is there any way we can have a safety check? This is honestly the end-users responsibility not to mess up, but are there any attributes on the analyzer that could say help make sure the recording being temporarily used is reasonable?

zm711 avatar May 02 '24 10:05 zm711

I really like this idea because we often do the analysis and then move everything over to our server for long-term storage which means the recording-analyzer link gets broken. My only concern (and this is now from experience helping others) is that people forget what goes with what. So is there any way we can have a safety check? This is honestly the end-users responsibility not to mess up, but are there any attributes on the analyzer that could say help make sure the recording being temporarily used is reasonable?

I think that we could check that the recording has the correct recording attributes, i.e., channel_ids, locations, etc

alejoe91 avatar May 02 '24 11:05 alejoe91

@zm711 can you check this again after the last commit?

alejoe91 avatar May 14 '24 13:05 alejoe91

This is good by me once tests pass . Was @samuelgarcia going to do a final readthrough as well?

zm711 avatar Jun 18 '24 13:06 zm711