AllenNeuralDynamics / AllenNeuralDynamics/aind-dynamic-foraging-data-utils

create_df_events includes all acquisition information.

Open
#128 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
2
Forks
0
Avg merge
1d 10h
Merged PRs (30d)
3

Description

As it is presently written, create_df_events automatically reads all event information in the acquisition section of the NWB and excludes only specific FIB events.

I am in the process of making NWBs for ephys which include additional acquisition data not expected on other rigs, specifically sniffing data. As it is presently written, create_df_events automatically includes all sniffing events (a lot since it is a continuous signal!) in the resulting data frame.

Rather than explicitly exclude certain data (e.g. FIB), I would suggest that create_df_events specifically include the desired data (Licks, go Cue, etc; the list is short so it shouldn't be to hard!), so that additional acquisition devices can be added to rigs without needing to update the code each time. There could even be the option to modify the included list.

I am open to other suggestions and am happy to make this change, but wanted to get feedback before taking the time to do so.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.