AllenNeuralDynamics / AllenNeuralDynamics/aind-metadata-mapper

Update FIP mapper to use passed patch_cable to fiber connection info

Offen
#416 1 Kommentar 0 Reaktionen 1 zugewiesene Person Beansprucht von @dougollerenshaw Auf GitHub ansehen
Vorherrschende Sprache
Python
Sterne
0
Forks
2
Ø Merge
1 T. 7 Std.
Gemergte PRs (30 T.)
1

Beschreibung

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)

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.