Support SpikeTrainList object and deprecate RecordingChannel in Elephant

[mdenker]

Since Neo 0.10.0, there is a new SpikeTrainList object. While this new construct behaves like a list, there are a couple of instances where Elephant directly tests if an input is explicitly a list of SpikeTrains. This PR adds support for allowing SpikeTrainList objects as well as lists of SpikeTrains in these circumstances.

Fixes:

• conversion.BinnedSpiketrain (wtih regression test, where SpikeTrainLists would lead to incorrect binned matrices)
• spike_train_synchrony.Synchrotool class
• spade module
• statistics module, instantaneous_rate()

This PR fixes #431.

Also, in Neo 0.10.0, RecordingChannelGroup objects are no longer supported. This affects the current_source_density, where electrode coordinates where optionally extracted form this object. This PR fixes CSD methods by taking electrode positions from an annotation of the signal. As in the previous version of Elephant, CSD methods alternatively take electrodes as a direct parameter. By nature of the API break in Neo, this also necessarily breaks API in Elephant.

[pep8speaks]

Hello @mdenker! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

• In the file elephant/current_source_density.py:

Line 82:80: E501 line too long (80 > 79 characters)

• In the file elephant/test/test_spade.py:

Line 189:13: E122 continuation line missing indentation or outdented
Line 190:13: E122 continuation line missing indentation or outdented

Comment last updated at 2022-03-10 15:55:34 UTC