-
Notifications
You must be signed in to change notification settings - Fork 4
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
Brainwide Trial Data #26
Comments
I assume we will want to map the column names to NWB Best Practice convention First proposal for this is
|
Another thing to discuss would be that what gets returned by the trials query in their API is an unordered dictionary, and sometimes the keys are out of a standard order In NWB, the columns are defined in an ordered fashion, and we should fix the columns to a standard order for consistency between all the NWB files for this experiment First proposal for order of columns (independent of name mapping proposal above)
the thinking here is to cluster the timing columns from the others and within that cluster similar prefixes |
If we scan all trials tables in the entire IBL dataset via
we find that there are about 11 different configurations, and sometimes common fields for certain configurations have uneven shapes for their values (the ONE system outputs a
stderr
notification when this happens)While it might in theory be possible to define a global trials interface, it would seem better practice implement this more modularly for each experiment
If you limit the above snippet to just Brainwide sessions, they all share a single common configuration, minus the
goCueTrigger_times
they asked us not to includeFor the Brainwide trials data we have the custom columns (with
description
s used from https://docs.google.com/document/d/1OqIqqakPakHXRAwceYLwFY9gOrm8_P62XIfCTnHwstg/edit#)the shapes for each of these fields for each of the sessions in the Brainwide experiments have been verified to be consistent.
The text was updated successfully, but these errors were encountered: