[BEP032]: add microelectrode electrophysiology example dataset#527
[BEP032]: add microelectrode electrophysiology example dataset#527bendichter wants to merge 15 commits intobids-standard:masterfrom
Conversation
Add microephys dataset demonstrating extracellular (ecephys) and intracellular (icephys) recordings in NWB format to the dataset listing.
|
Hi Ben, thank you for contributing this! could you please rename the dataset directory to something more specific such that we can distinguish it from more future datasets? Perhaps, microephys_toy or anything else that is suitable? Linking my PRs here to the @bids-standard/bep032 branch for completeness:
|
…Allen CCF coords - Move ecephys metadata files from session to subject level directory - Remove session identifier from channels, electrodes, and probes filenames - Add Allen CCFv3 coordinate system JSON with spatial reference metadata - Add electrode positions table with 32 electrodes mapped to Allen CCF space
|
The good news is this almost works validating against the compiled BEP32 PR: bids-standard/bids-specification#1705. The bad news is it does not work 100% and I don't know why: % bst -v export --schema src/schema --output src/schema.json
% bids-validator -s file:///Users/bdichter/dev/bids-specification/src/schema.json /Users/bdichter/dev/bids-examples/microephys/
[ERROR] MICROEPHYS_SPACE_MISMATCH The space entity in electrodes.tsv must match the space entity in the
corresponding coordsystem.json file for microelectrode electrophysiology data.
/sub-mouse01/ecephys/sub-mouse01_space-AllenCCFv3_electrodes.tsv
Please visit https://neurostars.org/search?q=MICROEPHYS_SPACE_MISMATCH for existing conversations about this issue.
[ERROR] MICROEPHYS_ELECTRODES_SPACE_REQUIRED If a coordsystem.json file with a space entity is present for microelectrode
electrophysiology data, a corresponding electrodes.tsv file with the same
space entity must also be present.
/sub-mouse01/ecephys/sub-mouse01_space-AllenCCFv3_coordsystem.json
Please visit https://neurostars.org/search?q=MICROEPHYS_ELECTRODES_SPACE_REQUIRED for existing conversations about this issue.
[ERROR] EMPTY_FILE Empty files not allowed.
/sub-mouse01/ses-01/ecephys/sub-mouse01_ses-01_task-rest_ecephys.nwb
/sub-mouse01/ses-01/ecephys/sub-mouse01_ses-01_task-reach_ecephys.nwb
1 more files with the same issue
Please visit https://neurostars.org/search?q=EMPTY_FILE for existing conversations about this issue.
Summary: Available Tasks: Available Modalities:
24 Files, 15.6 kB Reach to Grasp microephys
2 - Subjects 1 - Sessions Resting State
IV Curve Characterization
If you have any questions, please post on https://neurostars.org/tags/bids.The empty files are not an issue-- this repo specifically requests them. The problem is the space validation. I'm pretty sure I added those correctly but they are not validating. @yarikoptic and @ree-gupta , you both know the schema validation stuff better than I do. Any ideas? |
- Add probes.json sidecar with NeuroNexus A1x32-Poly3-10mm-50-177 model description and reference URL - Add model column to probes.tsv - Update manufacturer from IMEC to NeuroNexus to match probe model
…-examples into bep32-microephys
- Rename microephys directory to microephys_toy for clarity - Add "Features Demonstrated" section highlighting key BIDS features: - Allen CCF v3 space registration - Multiple probes per subject - ProbeInterface library references - Custom probe definitions via BIDS URIs - Update dataset structure to document dual-probe setup (probe01/probe02) - Add customprobe1.json ProbeInterface definition for 16-channel probe
Add 16 new electrode entries (e033-e048) for probe02 targeting the right hemisphere CA1 hippocampal region with iridium contacts in the sub-mouse01 electrode coordinates file.
Hi @bendichter , thanks again. It was to do with the associations files. The original validation failed because the coordsystem association lacked Adding I also improved the comments in the schema files. I cloned your fork, and checked, it passes the validation now 🎉 |
|
yay! I can confirm the fix worked 💯 |
|
From my perspective this is ready |
I get a couple of errors:
|
|
Huh I didn't see these in the valuation errors |
Surprisingly the probes error went away when I used, |
|
ok I recompiled and now I am getting the errors you mentioned. I will let you know when they are all fixed. |
Move x, y, z columns adjacent to each other and relocate hemisphere column after z coordinate for better logical grouping in electrodes TSV.
…ufacturer Move the 'model' column from position 3 to after 'manufacturer' column, improving logical grouping of device-related metadata fields.
…dinates Move the hemisphere column from position 3 to position 6 in sub-mouse02_electrodes.tsv, placing it after the x, y, z coordinate columns for improved logical grouping of spatial information.
Move x, y, z columns together after probe_name and relocate hemisphere column after z coordinate for more logical data organization.
…ive atlas IDs - Add CogAtlasID references for task definitions - Convert PharmaceuticalName and PharmaceuticalDoseUnits from arrays to strings - Fix SampleEnvironment format: "in-vivo"/"ex-vivo" → "in vivo"/"ex vivo" - Add missing RecordingDuration field to IVcurve task metadata
|
@ree-gupta ok, fixed! |
|
We need to remove |
|
Note that I have added testing against our BEP032 PR schema changes. Unfortunately testing fails |
|
Also take a look here: #491 - it is reviewed and complete and passes the validator. |
- Add required electrode_name column to channels.tsv (ecephys and icephys) - Remove deprecated dimension_unit column from probes.tsv - Convert probe width/height/depth values from um to mm - Remove optional reference column from channels.tsv Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
@bendichter @ree-gupta what would you like to do with this one? Looks like there are conflicts to resolve in order to retrigger dev tests against the BEP branch |
This example dataset demonstrates the BIDS microelectrode electrophysiology
(microephys) specification for both extracellular (ecephys) and intracellular
(icephys) recordings.
Features Demonstrated
This dataset demonstrates the following key features:
Registration to Allen CCF v3 space: Electrode positions are provided in
Allen Common Coordinate Framework v3 (see space-AllenCCFv3 files)
Multiple probes in a single subject: sub-mouse01 uses two probes
simultaneously (probe01 and probe02) targeting different brain regions
Probe defined in ProbeInterface Library: probe01 uses the
A1x32-Poly3-10mm-50-177 model from the ProbeInterface library, referenced
via TermURL in the probes.json sidecar
Probe defined using a custom ProbeInterface definition: probe02 uses a
custom probe definition stored in the probes/ directory, referenced via
BIDS URI (bids::probes/customprobe1.json)
added by @yarikoption now that #515 is merged: