visualizations.raster improvements

[lmcintosh]

Would be nice if this were simpler; i.e. just take an (n,2) array of spike times and cell / trial ID

[leongjcs]

I agree.

On Tue, Jul 7, 2015 at 2:22 PM Lane McIntosh notifications@github.com
wrote:

Would be nice if this were simpler; i.e. just take an (n,2) array of spike
times and cell / trial ID

—
Reply to this email directly or view it on GitHub
#54.

[nirum]

we should split the current code in visualizations.raster into two functions, one that splits a list of spike times into multiple trials given a trial length, and then the raster function should just take something like a two-column ndarray where the first is the spike times and the second is a label or index for that spike time.

e.g.:

raster(spk_array):
   plt.plot(spk_array[:,0], spk_array[:,1], '.');

This lets you use raster if you want to plot a raster across cells instead of trials, for example.

[nirum]

Thoughts, @bnaecker? This is a breaking change, but makes things simpler. Check out the modifications in the above commit