search

catalystneuro / mease-lab-to-nwb

MIT License
3 stars 2 forks source link

multirecording = se.MultiRecordingTimeExtractor(recordings) - IndexError: index 107136280 is out of bounds for axis 0 with size 107136280 #37

Closed ross-folkard closed 3 years ago

ross-folkard commented 3 years ago

Hi @luiztauffer @alejoe91 . I am trying to concatenate 2 files and am getting this error message. Might this also be due to the time-offset differences betwen intan and CED? image

alejoe91 commented 3 years ago

@rf13734 I'll check this out! Probably this happens when concatenating 2 recordings with timestamps (like CED ones).

In the meanwhile, this should fix the resample issue: https://github.com/SpikeInterface/spiketoolkit/pull/485

alejoe91 commented 3 years ago

@rf13734 this should be fixed on spikeextractors master: https://github.com/SpikeInterface/spikeextractors/pull/641/commits/a941fb1642faccf9bc4398d03b1b9bf9a5b8fde4

Can you test installing both spikeextractor and spiketoolkit from the official master? both this and the resampling problem shoudl be fixed

ross-folkard commented 3 years ago

@alejoe91 I tried this and now have a new error for self.set_channel_gains during the recording = se.CEDRecordingExtractor(ced_file, smrx_channel_ids=rhd_channels)

image

image

alejoe91 commented 3 years ago

@rf13734 spikeextractors is not installed from master. You have to to the clone + uninstall + python setup.py develop trick with this repo: https://github.com/SpikeInterface/spikeextractors

ross-folkard commented 3 years ago

@alejoe91 Ah yes, sorry!

ross-folkard commented 3 years ago

@alejoe91 Thanks, now installed the new spiketoolkit and spikeextractors (I think) image

But still getting a 30kHz sampling rate

image

and for recording_processed too image

alejoe91 commented 3 years ago

Yes that's not in master yet. or is it @luiztauffer ??

ross-folkard commented 3 years ago

@alejoe91 ah okay, I'll carry on through the pipeline, but so far so good!! Thanks!

luiztauffer commented 3 years ago

it is ready to come here

ross-folkard commented 3 years ago

@luiztauffer However, when I swap out the file you used for the same file in my computer, I no longer get the corrected sampling frequency.

image

image

image

image

Apologies, this was posted in the wrong issue

luiztauffer commented 3 years ago

are you sure you're checked out in the PR version?

luiztauffer commented 3 years ago

sorry for all these versioning headaches... they will cease once all changes get merged in their respective repos and we make the releases

ross-folkard commented 3 years ago

are you sure you're checked out in the PR version?

@luiztauffer Ah is this PR version the new version of fix_ced_2?

image

I'll install this and try it out

ross-folkard commented 3 years ago

Works now! Thanks again!

image