Skip to content
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

Incorrect number of trials for RVG16_B01 #114

Closed
jthermiz opened this issue Sep 10, 2021 · 2 comments · Fixed by #130
Closed

Incorrect number of trials for RVG16_B01 #114

jthermiz opened this issue Sep 10, 2021 · 2 comments · Fixed by #130

Comments

@jthermiz
Copy link
Member

There are only 118 trials list for /clusterfs/NSDS_data/nwb/test/RVG16/RVG16_B01.nwb but there should be 120 (60 for stimulus and 60 for baseline).

image

@JesseLivezey
Copy link
Member

Another stim parsing problem for a tone150 block:

ValueError: ToneTokenizer: Incorrect number of stimulus onsets found in block RVG06_B06. Expected 4500, found 4496.

@jihyunbak
Copy link
Member

We were missing the first mark in wn2 blocks - this can be fixed if a lower threshold was used (0.1 instead of 0.3), as implemented in PR #130. We had the same observation before (for example #102), but we understand a little better now: the first mark is smaller than all the other markers in the recorded mark time series.

image

This is probably because the mark for wn2 starts right at the beginning of the audio file (although not sure exactly why)...

image

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 a pull request may close this issue.

3 participants