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

Can't assign different units to behavioral events #19

Open
JoshiAbhilasha opened this issue Mar 10, 2023 · 1 comment
Open

Can't assign different units to behavioral events #19

JoshiAbhilasha opened this issue Mar 10, 2023 · 1 comment
Assignees
Milestone

Comments

@JoshiAbhilasha
Copy link

JoshiAbhilasha commented Mar 10, 2023

We currently do not have the ability to assign units to Behavioral Events (such as for Din, Dout, Acceleration, and Gyroscope data). Our metadata files do have the Units section but we currently leave it as 'unspecified'. As far as I understand, even if unit information was provided, it does not populate downstream in the .nwb files.

Suggested modification: Assign a unit to each type of 'behavioral_event' in the metadata. For example:

  • Din and Dout: the values lie between 0 and 1 and 'unspecified' is a reasonable choice of unit
  • Accelerometer: spike-gadgets headstages use +/- 2G full range with an in-practice resolution per step 2/32767 g (0.000061 g). The unit should be 'g' and we would need to multiply the recorded values with 0.000061 g.
  • Gyroscope: spike-gadgets headstages use +/- 2000 degrees per turn (dps) with an in-practice resolution of 0.061 deg/sec. The unit should be 'd/s' and we would need to multiply the recorded values with 0.061 deg/sec.

and ensure that this information is populated downstream in the .nwb files.

@edeno edeno self-assigned this Mar 29, 2023
@edeno edeno transferred this issue from LorenFrankLab/rec_to_nwb Jul 11, 2023
@edeno edeno assigned samuelbray32 and unassigned edeno Jul 11, 2023
@edeno edeno assigned rly and unassigned samuelbray32 Aug 1, 2023
@edeno
Copy link
Collaborator

edeno commented Sep 29, 2023

Should separate analog data into separate nwb time series for each type of analog signal.
Should also name the Ain1, etc. in the yaml so people know what they are.

@edeno edeno added the enhancement New feature or request label Oct 20, 2023
@edeno edeno added this to the v0.1.0 milestone Oct 25, 2023
@edeno edeno modified the milestones: v0.1.0, 0.2.0 Oct 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants