Skip to content

Conversation

@neuromusic
Copy link
Contributor

No description provided.

@neuromusic neuromusic changed the title missing intervalsms workaround [WIP] missing intervalsms workaround Jul 19, 2018
@neuromusic
Copy link
Contributor Author

ok this PR is ready to review. it should do the following...

I would appreciate reviews & try-it-out testing from others (on the pkl files in #226) before we merge it.

cc @dougollerenshaw @Ryval @nicain @derricw

@neuromusic neuromusic changed the title [WIP] missing intervalsms workaround missing intervalsms workaround Jul 20, 2018
@dougollerenshaw
Copy link
Contributor

Great! Taking a look right now.

@dougollerenshaw
Copy link
Contributor

dougollerenshaw commented Jul 20, 2018

This looks good to me. I was able to open one of the previously failing files and make a summary plot. Behavior looks exceptionally good for this mouse on this day:
image
The only red flag I can see is when I plot running data. There are some big negative transients in the running speed plot:
image
I haven't been looking at the running speed regularly enough to know whether these transients are happening regularly. But they're almost certainly not representative of the animals actual running behavior. Could this have something to do with the missing data in the file?

@dougollerenshaw dougollerenshaw self-requested a review July 20, 2018 19:50
@neuromusic neuromusic merged commit 8f9d28d into 0.4.x Jul 20, 2018
@neuromusic neuromusic deleted the fix/running branch July 20, 2018 21:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants