Decide and test priority of group_name and group in /tools/spikeinterface/spikeinterface.py functions.

[h-mayorquin]

From here:
#634

The relevant comments are here:

#634 (comment)

And more specifically:
@alejoe91

When loading and setting a probe on the SI side, the group property is set. However, the add_electrode_groups has a precedence on the group_name over the group property to define electrode groups (see here). Therefore, the probe group is ignored.

So if the recorder of the user has a group_name our functions will write that one instead of the group from the probe which is probably what the user intended.

I need to check a couple of cases on the spikeinterface side. I will come back to this but I wanted to document this so we don't forget.