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

create_df_events includes all acquisition information.

未关闭
#128 1 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
主要语言
Python
星标
2
派生
0
平均合并
1 天 10 小时
30 天内合并 PR
3

描述

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.

贡献指南

这个仓库没有索引到贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。