AllenNeuralDynamics / AllenNeuralDynamics/comb

Reconsider stimulus_presentations with gray screen and movie

Open
#26 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
1
Forks
0
PR merge metrics
No merged PRs in 30d

Description

**Is your feature request related to a problem? Please describe.**
stimulus_presentations now include gray screen and movie, but while implementing it there has been changes in other parts of the stimulus_presentations, such as 'omitted' and 'image_name'. Most of these were filled with NaN, which breaks downstream analysis codes such as GLM

**Describe the solution you'd like**
I think there are 2 options:
1. Having a separate dataFrame for including the full time series
2. Changing NaN to other values (this requires more discussion on which analysis codes are going to be impacted).

**Describe alternatives you've considered**
A clear and concise description of any alternative solutions or features you've considered.

**Additional context**
Add any other context or screenshots about the feature request here.

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.