search

baccuslab / pyret

Python tools for analysis of neurophysiology data
https://pyret.readthedocs.io/en/master/
MIT License
35 stars 8 forks source link

visualizations.raster improvements #54

Closed lmcintosh closed 9 years ago

lmcintosh commented 9 years ago

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

leongjcs commented 9 years ago

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 https://github.com/baccuslab/pyret/issues/54.

nirum commented 9 years ago

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 commented 9 years ago

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