AllenNeuralDynamics / AllenNeuralDynamics/aind-metadata-mapper
Update FIP mapper to use passed patch_cable to fiber connection info
- Dominant language
- Python
- Stars
- 0
- Forks
- 2
- Avg merge
- 1d 7h
- Merged PRs (30d)
- 1
Description
Currently the FIP mapper makes assumptions about the patch_cord to fiber connection scheme. See the detailed issue description here: https://github.com/AllenNeuralDynamics/Aind.Physiology.Fip/issues/40.
Ultimately, it'd be preferable for this information to be specified in the extracted fiber metadata. After:
* https://github.com/AllenNeuralDynamics/Aind.Physiology.Fip/issues/40
* https://github.com/AllenNeuralDynamics/aind-metadata-extractor/issues/35
are resolved, we should update the FIP mapper to:
* Remove the hard-coded assumptions about the patch_cord to fiber mapping
* Verify that all referenced fibers exist in the procedures for the given subject
* Verify that the number of active patch cords is less than or equal to the number of implanted fibers
* Verify that no duplicate mappings exist (i.e. each patch cord maps to only one fiber)
Contributor guide
Assessment
This issue has not been assessed yet.