Skip to content

Add Support for reading NWB files#261

Merged
pauladkisson merged 28 commits intomainfrom
read_nwb
Apr 13, 2026
Merged

Add Support for reading NWB files#261
pauladkisson merged 28 commits intomainfrom
read_nwb

Conversation

@pauladkisson
Copy link
Copy Markdown
Collaborator

@pauladkisson pauladkisson commented Apr 7, 2026

Fixes #88

Replaces #186

  • ndx-events v0.2
  • ndx-events v0.4
  • ndx-fiber-photometry v0.1
  • ndx-fiber-photometry v0.2

This PR also fixes a pickling issue for long storenames in read_and_save_all_events revealed by the mock NWB file.

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 13, 2026

Codecov Report

❌ Patch coverage is 92.97297% with 13 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/guppy/extractors/nwb_recording_extractor.py 93.60% 4 Missing and 7 partials ⚠️
src/guppy/orchestration/read_raw_data.py 80.00% 1 Missing ⚠️
src/guppy/orchestration/storenames.py 75.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@pauladkisson pauladkisson marked this pull request as ready for review April 13, 2026 18:30
@pauladkisson pauladkisson mentioned this pull request Apr 13, 2026
3 tasks
@pauladkisson pauladkisson merged commit f4cd083 into main Apr 13, 2026
11 checks passed
@pauladkisson pauladkisson deleted the read_nwb branch April 13, 2026 20:00
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.

1 participant