Add apply curation to gui

[chrishalcrow]

Allows the user to apply the current curation to the analyzer, creating a new analyzer in memory.

What this PR does:

• When the user clicks the apply_curation button, we create a new, curated analyzer in memory. All views are _reinitialized and the CurationModel is created and stored in controller.applied_curations and the current curation is cleared. Now the user can start curating again.
• Adds _reinitialize method to all Views, allowing the view to update when the analyzer is updated (might also be useful for re-computations!)
• Allows the user to save multiple curations as a SequentialCuration.
• To use this mode, the user must pass the iterative_curation flag to the cli or the run_mainwindow function.

I think this is pretty much done!

To do:

• add to panel
• save original analyzer without extensions somewhere
• keep track of the chain of curations (must add new_unit_ids)
• ensure properties are properly kept track of
• ensure colours are properly kept track of
• be able to save multiple curations
• allow user to undo apply

[samuelgarcia]

Clearly I totaly I lost the control here.
My only wish now is that this will be an optional (and maybe hidden) feature that will be false by default.

[chrishalcrow]

Haha, I don't think so Sam! The core logic has changed much - most of the gross code is related to storing the chain of curations and being able to save them!

[chrishalcrow]

add comment

[chrishalcrow]

@alejoe91 @samuelgarcia I think this is ready!

[alejoe91]

@chrishalcrow I tested this and added a Restore original option (only original, no undo). In conjunction with SpikeInterface/spikeinterface#4718, it works perfectly!

[alejoe91]

@chrishalcrow done! fixed the QT selection and added CurationView settings mapping the apply_curation args (only if --iterative)

Ready to merge on my end!

[alejoe91]

Note that this is needed to avoid failures due to all non-mergeable units: SpikeInterface/spikeinterface#4804