Testing pot_auto_merge in gui raises error

[zm711]

I just used get_ground_truth_recording to make an analyzer: got this error

  File ~\Documents\GitHub\spikeinterface-gui\spikeinterface_gui\pairlist.py:270 in compute
    self.pairs, self.merge_info = self.controller.compute_auto_merge(**params)

  File ~\Documents\GitHub\spikeinterface-gui\spikeinterface_gui\controller.py:405 in compute_auto_merge
    potential_merges, extra = get_potential_auto_merge(

TypeError: get_potential_auto_merge() got an unexpected keyword argument 'minimum_spikes'

I think after Pierre and Aurelien updated that this might take an update.

[FrancescoNegri]

Yeah, I encountered the same error, the parameters passed to get_potential_auto_merge() from the GUI are not the expected ones.
I could try to fix the auto merge dialog with the current SpikeInterface parameters, opening a pull request if this issue has not been solved yet.

[zm711]

It has not @FrancescoNegri. I would say go ahead and do it. I keep planning on doing it, but it is low on my priority list and I think on Sam's too.

[samuelgarcia]

Hi
after this is merged (normally tomorow) SpikeInterface/spikeinterface#3435
Il will fix it on sigui side.

[samuelgarcia]

I merge today a PR and this should be fixed.