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

NWB to dataframe tools #26

Merged
merged 29 commits into from
Aug 9, 2024
Merged

NWB to dataframe tools #26

merged 29 commits into from
Aug 9, 2024

Conversation

alexpiet
Copy link
Collaborator

@alexpiet alexpiet commented Aug 2, 2024

Creates a tidy dataframe of the events in acquisition, ignoring FIP data.

  • resolves NWB to trials_table, and events table #1
  • resolves Add one new NWB file for testing #32
  • reorganizes files into alignment and nwb_utils
  • Updates pyproject.toml to add new hdmf_zarr stuff
  • ensures that df_trials and df_session works with new NWB format. This involves some hacky code to make metadata information consistent. This needs to be planned out - What data do we want in df_session?
  • Adds nwb_utils.create_fib_df
  • We need to add unit testing for new NBW files, but I want to merge this in.

@alexpiet alexpiet requested a review from rachelstephlee August 2, 2024 21:30
@alexpiet
Copy link
Collaborator Author

alexpiet commented Aug 5, 2024

@rachelstephlee I think this is ready for review and merging in. There is more work to do, but its already getting so big.

@alexpiet alexpiet changed the title creating events table NWB to dataframe tools Aug 8, 2024
Copy link
Collaborator

@rachelstephlee rachelstephlee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good. thanks for all your work on this!

@alexpiet alexpiet merged commit 194861f into main Aug 9, 2024
3 checks passed
@alexpiet alexpiet deleted the add_lick_table branch November 13, 2024 03:37
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 this pull request may close these issues.

Add one new NWB file for testing NWB to trials_table, and events table
2 participants