Alessio Buccino

@samuelgarcia added the sparse slicing. Ready to review

@jonahpearl We actually have some code lying around for this: https://github.com/AllenNeuralDynamics/aind-ephys-ibl-alignment-gui-converter/blob/main/code/run_capsule.py

Not sure honestly, but maybe @jsiegle can comment on that? I think that your solution should be correct. The extremum channel index is already the channel that should be used.

> Would there be a release on probeinterface? If not, could we also add this so we can pin neuroconv to the incoming release: > > #2833 > > Also,...

> 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....

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

Hi Domi, I think that the problems i that you're getting `NaN` estimates of unit locations, since the error comes from ```x, y = unit_location[0], unit_location[1] --> spikeinterface/src/spikeinterface/widgets/unit_summary.py:107) ax1.set_xlim(x -...

A note on your note: you can also do: ``` recording_f = spre.bandpass_filter(recording) ... we.set_recording(recording_f) ``` This will NOT save the filtered data, but filter on the fly!

Hi @RobertoDF The quality metrics are computed as an extension, but the are not set as properties to the sorting objects. In order to add `amplitude_cutoff` (or any other QM)...

@h-mayorquin I'm taking another look at this and it seems in good shape. Maybe we should add an option to the compare_* function to select the matching strategy?