hasselmonians / cmbhome Goto Github PK

To replicate, open visualize and check gridness autocorr before checking a gridness score. Two possible solutions:

1. Remove this option. The actual plot differs whether you're using Gridness2/3 (and the callback for those do the plotting)
2. Affect whether Gridness2/3 actually generates the donut RM (currently, they do, and there's no way around it)

Hey, thanks for the nice package - it looks like it is going to be useful for visualizing and analyzing my grid cell simulation data.

I'm trying to get the scale and orientation of several grid cell receptive fields. I noticed that the gridDistance method on Session is broken - it uses the wrong row and column indices due to rowInd and colInd being indexed by a permutation vector more than once, putting them in the wrong order.

It would also seem more standard to use the center peak rather than the maximum peak to compute grid scale and orientation (avoiding edge effects).

Has this function been replaced with a different one that I should use instead? If not I'd be glad to make some fixes and submit a pull request!

Currently ignores input options for method of phase estimation. Also, only the wavelet method seems to be implemented.

Currently LFP.myvar gets affected by epochs, but there's no easy way to cross reference spike times and LFP time-scale variables. Currently implement as:

root.lfp.b_myvar(root.spike(root.cel(1),root.cel(2)).i_lfp)

But, is tedious and doesn't apply epoching. Implement as Session.cel_lfpmyvar, make sure to apply epoching and active cell.

• Currently the units in the toolbox are a mish-mash (eg: root.vel is in pixels/second, and some functions request thresholds in cm/second and other pix/second).
• Add a flag field for units (frames/seconds, pixels/cm). Store raw data as whatever is given from the raw data, and multiply by the spatial_scale / fs_video accordingly when variable is called.
• Functions taking in units should respect the current state, and when possible use the units in plotting as well, to remove ambiguity

Replicate something to the effect of CMBHOME.SEEscript for axona import. Include merging in of AxonaImport code.