-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support for ndx-events.Events and pynwb.behavior.BehavioralEpochs #168
Comments
Has there been any progress on this? Are there any blockers I can help with? Also a note: |
Hi @pauladkisson I've been working on some other things so probably won't get to this for a little while. Do you know typescript/react? I can point you to the relevant code if you want to take a look. |
Unfortunately I do not. But thank you for the update. |
Could you describe what the views should look like? For processing/behavior/behavioral_epochs/reward_port_intervals |
I would suggest an analog to matplotlib.pyplot.eventplot for the Events and filled region for the IntervalSeries. For example, see my example notebook. |
Bumping this bc my current conversion also needs ndx-events.Events visualization: https://neurosift.app/?p=/nwb&url=https://api.dandiarchive.org/api/assets/7f969b84-db9b-48ab-8935-b1ca21f5b607/download/&dandisetId=001259&dandisetVersion=draft |
@pauladkisson I think that Dandiset is embargoed. Would you be able to give me access? |
Yes, you are right it is embargoed. You should have access now. |
@pauladkisson I have added a simple view plugin for the ndx-events/Events type. Let me know what the next iteration should be. |
Events look good to me! |
Not strictly related to this issue, but what is going on with the encoder time series plot: https://neurosift.app/?p=/nwb&url=https://api.dandiarchive.org/api/assets/7f969b84-db9b-48ab-8935-b1ca21f5b607/download/&dandisetId=001259&dandisetVersion=draft&tab=neurodata-items:neurodata-item:/processing/behavior/targetOUT%7CEvents@neurodata-item:/processing/behavior/toneIN%7CEvents@neurodata-item:/processing/behavior/toneOUT%7CEvents@neurodata-item:/processing/behavior/valve%7CEvents I'm not seeing any time series, which is especially weird since the lick time series viz seems to be working just fine: https://neurosift.app/?p=/nwb&url=https://api.dandiarchive.org/api/assets/7f969b84-db9b-48ab-8935-b1ca21f5b607/download/&dandisetId=001259&dandisetVersion=draft&tab=neurodata-items:neurodata-item:/processing/behavior/targetOUT%7CEvents@neurodata-item:/processing/behavior/toneIN%7CEvents@neurodata-item:/processing/behavior/toneOUT%7CEvents@neurodata-item:/processing/behavior/valve%7CEvents |
I think it's probably due to some NaN's that NS is choking on. Perhaps open a separate issue for that? |
See new issue here: #219 |
So ndx-events.Events works now, but pynwb.behavior.BehavioralEpochs is still broken. Would you like me to close this issue and open up a new one just for pynwb.behavior.BehavioralEpochs? |
That would be helpful thanks. |
See new issue: #220 |
See https://neurosift.app/?p=/nwb&url=https://api.dandiarchive.org/api/assets/7d1990d7-a714-406e-9cb2-1b8f4015869e/download/&dandisetId=000971&dandisetVersion=draft
for events
and https://neurosift.app/?p=/nwb&url=https://api.dandiarchive.org/api/assets/7d1990d7-a714-406e-9cb2-1b8f4015869e/download/&dandisetId=000971&dandisetVersion=draft
for BehavioralEpochs
The text was updated successfully, but these errors were encountered: